Red Hat Bugzilla – Bug 1007290
Missing 'save' button under content-view filters --> 'Repositories' tab
Last modified: 2014-04-24 13:07:53 EDT
Created attachment 796694 [details]
missing save button under content-view filters --> 'Repositories' tab
Description of problem:
The 'save' button is missing under 'Repositories' tab of content-view filters.
I think its because of new addition of Puppet Repository' panel.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. Create a content-view
2. create a filter
3. Go to repositories tab
Missing Save button
Save button should be there.
Sachin, I saw that yesterday and it seems that it the change was done on purpose. As you mentioned before, your selection persists as you switch between tabs. Need to verify with DEV and if this is the design, then we need to update our code.
In case the change is as per design then UI should raise a notification on selecting a product or repo.
This is a change that was introduced by https://github.com/Katello/katello/pull/2870 . Rather than require the user to click 'save' the product/repo selector was updated to use ajax to perform an immediate update.
Regarding notifications. Currently we are showing a spinner that disappears when the product/repo is saved. We also log a notification to the database but we don't show it to the user.
I tried turning on notifications but it looks like there is some type of bug (Justin confirmed this) where the code checks the notification type and clears out notifications of the same type. I think this was to reduce dupe errors. The problem is that it only shows one notice if you update the definition 2+ times in quick succession.
I could fix this but then the user would probably be bombarded with notices depending on how fast they update the definition. Also, I'm not sure what sort of regressions I might introduce.
Another alternative would be maybe to show some sort of indicator on the page that the definition was updated (as opposed to a notice). The downside to this would be that it would be inconsistent with the rest of the app and it'll be temporary as the page will be redone soon with the nutupane changes.
Does anyone have any ideas or preferences?
Another option would be to go back to a single save button and just show a notice once it's clicked?
I need the notification so as to ensure(while automating test) whether the repo/product is properly selected.
Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.
Suchin, I tried to contact you on IRC to test out the success notification stuff. You should now see a green check mark if the definition gets updated successfully. It'll disappear after 3 seconds or if there's activity. Please test it out (once it gets merged) and let me know if you have any feedback.
Verified with snap6 compose2 (MDP2)
We have now + Add button to select the repositories. When user select it and click on 'Add' , a green check mark appears which disappears after few secs (as mentioned in comment13)
Created attachment 814523 [details]
green check-mark 'Saved' appears as success
This was verified and delivered with MDP2. Closing it out.