make updates2stable only show on stream page, and link it to the user of the stream, rather than a config value.
LGTM
Do you think it's better to have it show "no updates ready for stable" rather than just not show the widget at all? I thought it would be more practical if widgets who have nothing to say don't clutter up the page, it would allow adding more widgets even if they are rarely useful.
I'm probably not the best person to ask about UX stuff. Hiding it when it has nothing to say sounds interesting enough though. @ryanlerch what do you think?
@abompard for me it felt a little strange to completely hide the widget -- i at first thought it was a bug when I added the widget, and then it wasnt appearing in the UI. I only figured out that this was expected behaviour after looking at the code.
I wonder if @mizmo has any thoughts on this one.
THe two behaviours being discussed here are:
or
rebased onto 7252b221a88a9c2bbfb7d959e010b4d3547e5c2d
Since this widget was previously doing the hiding functionality, i have reverted that part of my patch, and now this patch just rescrticts this widget to the stream page, and uses the user's name as the parameter.
I have filed this issue to discuss the hiding / not hiding of widgets:
https://pagure.io/fedora-hubs/issue/512
cheers, ryanlerch
Pull-Request has been merged by abompard
make updates2stable only show on stream page, and
link it to the user of the stream, rather than a
config value.