Bug 1608362 - [downstream clone - 4.2.5] [RFE] Disable Event notification popup in admin portal
Summary: [downstream clone - 4.2.5] [RFE] Disable Event notification popup in admin po...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 4.2.1
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ovirt-4.2.5
: ---
Assignee: Greg Sheremeta
QA Contact: Pavel Novotny
URL:
Whiteboard:
Depends On: 1590202
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-07-25 11:57 UTC by RHV bug bot
Modified: 2021-09-09 15:12 UTC (History)
14 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
This update adds a feature to control pop up notifications. Once 3 or more notifications are showing, "Dismiss" and "Do not disturb" buttons will appear that allow the user to silence notifications.
Clone Of: 1590202
Environment:
Last Closed: 2018-07-31 17:50:11 UTC
oVirt Team: UX
Target Upstream Version:
Embargoed:
lsvaty: testing_plan_complete-


Attachments (Terms of Use)
Do Not Disturb - distant dropdown (198.21 KB, image/png)
2018-07-30 21:36 UTC, Pavel Novotny
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-43565 0 None None None 2021-09-09 15:12:27 UTC
Red Hat Knowledge Base (Solution) 3490061 0 None None None 2018-07-25 11:58:12 UTC
Red Hat Product Errata RHBA-2018:2318 0 None None None 2018-07-31 17:50:30 UTC
oVirt gerrit 92443 0 master MERGED webadmin: notifications drawer 2020-05-07 23:12:14 UTC
oVirt gerrit 92896 0 ovirt-engine-4.2 MERGED webadmin: notifications drawer 2020-05-07 23:12:14 UTC

Description RHV bug bot 2018-07-25 11:57:02 UTC
+++ This bug is a downstream clone. The original bug is: +++
+++   bug 1590202 +++
======================================================================

Description of problem:

There are multiple popup coming for the event notification in RHV portal. Wen there is any event happene at that time popup will come in admin portal. 
Currently there is no way to disable this popups, we want feature to disable this event notification popup in RHVM portal. 

Version-Release number of selected component (if applicable):

Red Hat Virtualization 4.2

How reproducible:

100%

Steps to Reproduce:
1. If there is an event then popup will come into RHV admin portal.


Actual results:

Popup coming for the events as notification box in RHV portal.

Expected results:-

There should be some way to disable this event notification popup.


Additional info:-

NA.

(Originally by Vaibhav Pagar)

Comment 1 RHV bug bot 2018-07-25 11:57:10 UTC
Created attachment 1450445 [details]
Hidden content on a form

Here is an example of the issue. In some cases when the environment is busy there is almost no chance to remove these popups and it prevents the user from using the portal.

(Originally by Roman Hodain)

Comment 3 RHV bug bot 2018-07-25 11:57:16 UTC
Definitely an issue, and I can think of a few ways to solve from a UX perspective.

Liz, can you provide your opinion?

(Originally by Greg Sheremeta)

Comment 4 RHV bug bot 2018-07-25 11:57:22 UTC
I agree with Greg, not necessarily disable completely, but do some UX magic to make it more consumable and less disturbing.

(Originally by Tomas Jelinek)

Comment 6 RHV bug bot 2018-07-25 11:57:32 UTC
Hi all,

I think allowing the user to disable notifications as a feature, close all, or snooze are features that are avoiding the current problem. It would be great if we could pick up on the situations where we could let the user know in "bulk" that a number of things have happened. In this specific case, it would be really nice for the notification to be "6 VMs have been removed...". The user wouldn't feel like they've gotten a storm of notifications and it would scale to 100s of notifications if needed. Would it be technically possible to lump these together into one notification?

In some cases, the application might be experiencing the same issue over and over in which case it would be great to identify if that's happening and just notify the user once rather than with multiple. (This could be a case for allowing an action of "Don't notify me again about this"). Here's an example of ManageIQ handling this situation:
https://github.com/ManageIQ/manageiq-ui-service/pull/751

I think if we come across a use case where it does make sense to list out each toast notification individually, we should consider a "Clear All" feature, but it would be good to understand if there truly is a use case for this, or if we just need to be careful and considerate about which actions might cause a number of notifications at once.

What do folks think?

Thanks,
Liz

(Originally by Liz Blanchard)

Comment 7 RHV bug bot 2018-07-25 11:57:37 UTC
(In reply to Liz from comment #5)
> Hi all,
> 
> I think allowing the user to disable notifications as a feature, close all,
> or snooze are features that are avoiding the current problem. It would be
> great if we could pick up on the situations where we could let the user know
> in "bulk" that a number of things have happened. In this specific case, it
> would be really nice for the notification to be "6 VMs have been
> removed...". The user wouldn't feel like they've gotten a storm of
> notifications and it would scale to 100s of notifications if needed. Would
> it be technically possible to lump these together into one notification?
> 
> In some cases, the application might be experiencing the same issue over and
> over in which case it would be great to identify if that's happening and
> just notify the user once rather than with multiple. (This could be a case
> for allowing an action of "Don't notify me again about this"). Here's an
> example of ManageIQ handling this situation:
> https://github.com/ManageIQ/manageiq-ui-service/pull/751
> 
> I think if we come across a use case where it does make sense to list out
> each toast notification individually, we should consider a "Clear All"
> feature, but it would be good to understand if there truly is a use case for
> this, or if we just need to be careful and considerate about which actions
> might cause a number of notifications at once.
> 
> What do folks think?
> 
> Thanks,
> Liz

1. We might need some event supression mechanism. It sometimes may not make sense to output the same / similar event to the user multiple times in succession.
2. Do events by other users pop up?
3. I wonder if we should not pop up events for items originated from the API (not sure how feasible this is!)
4. Perhaps just alerts/warnings/async tasks?

(Originally by Yaniv Kaul)

Comment 9 RHV bug bot 2018-07-25 11:57:48 UTC
@QE, to test, you'll need to do a mass operation that will cause 3 or more toast notifications to occur within 8 seconds.

(Originally by Greg Sheremeta)

Comment 10 RHV bug bot 2018-07-25 11:57:54 UTC
WARN: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{'rhevm-4.2.z': '?'}', ]

For more info please contact: rhv-devops: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{'rhevm-4.2.z': '?'}', ]

For more info please contact: rhv-devops

(Originally by rhv-bugzilla-bot)

Comment 12 Pavel Novotny 2018-07-30 18:38:39 UTC
Verified in
ovirt-engine-4.2.5.2-0.1.el7ev.noarch
ovirt-engine-webadmin-portal-4.2.5.2-0.1.el7ev.noarch

There are now two new buttons for supression of multiple notifications:
1. Dismiss All
2. Do Not Disturb: for X minutes/hours/days / until Next Log In

All of them worked as expected.


One small difference between the design screenshot https://i.imgur.com/BOPonyM.png
is the position of the DnD options, see my attached screenshot.
The options are not located close to the DnD button but at the end of the notifications. Same in Firefox and Chrome.

@Greg, is it intentional or should I file new BZ to fix the positionning?

Comment 13 Greg Sheremeta 2018-07-30 18:55:48 UTC
The dropdown should appear right under the button, just like that imgur link. (I don't see another screenshot.)

If you cleared your cache and you still see that, yes please file a bug :)

Comment 14 Pavel Novotny 2018-07-30 21:36:29 UTC
Created attachment 1471645 [details]
Do Not Disturb - distant dropdown

Right, sorry, I forgot to attach it, here it is :)

Clearing the cache does not help. Filed bug 1609947.

Comment 16 errata-xmlrpc 2018-07-31 17:50:11 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:2318


Note You need to log in before you can comment on or make changes to this bug.