Bug 769619 - Notification on RH repo select/deselect is laggy and superfluous. Only show failed notifications.
Summary: Notification on RH repo select/deselect is laggy and superfluous. Only show f...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified vote
Target Milestone: Unspecified
Assignee: Jason E. Rist
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks: katello-blockers
TreeView+ depends on / blocked
 
Reported: 2011-12-21 13:44 UTC by Corey Welton
Modified: 2019-09-26 13:27 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-22 18:15:39 UTC
Target Upstream Version:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Corey Welton 2011-12-21 13:44:06 UTC 
Description of problem:
Not sure it is necessary to show a green notification everytime a RH is enabled/disabled. It feels sort of laggy, and I'm not sure it is necessary to show a notification every time for something that is basically a toggle.  

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


How reproducible:


Steps to Reproduce:
1.  Enable RH content
2.  Provider > Red Hat > Enable Repos
3.  Drill down into a product and select/enable/disable several repos in rapid succession
4. View UI
  
Actual results:
* A green "success" message pops up for each selection. The UI remains responsive. While there is a slight lag between select/deselect and the UI updating, this is reasonably acceptable. However the wait time between enabling/disabling and the subsequent success notification gives an impression of UI lagginess.
* User gets a success message for each select/deselect, which seems a bit excessive

Expected results:
* Don't show (log?) a success message each time a repo is enabled/disabled. Rather, only worry about errors and display/log them instead.

Additional info:
Comment 1 Jason E. Rist 2012-01-19 16:46:02 UTC 
commit 9457f9f165b94cd99b8b4dd9f14c3de9a2f8bc29
Author: Jason E. Rist <jrist>
Date:   Thu Jan 19 09:40:37 2012 -0700

    769619 - Fix for repo enable/disable behavior.
Comment 2 Corey Welton 2012-01-25 18:55:19 UTC 
That looks pretty good!  

QA Verified.
Comment 5 Mike McCune 2013-08-16 18:13:24 UTC 
getting rid of 6.0.0 version since that doesn't exist
Note You need to log in before you can comment on or make changes to this bug.