Launchpad itself

Subscribers portlet doesn't handle failure to load list well at all

Bug #386236 reported by William Grant
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Deryck Hodge
Launchpad itself 2.2.6

Bug Description

When a request to +bug-portlet-subscribers-content fails (due to a timeout, for example), the subscribers portlet is simply left mostly empty. There should be a message indicating that something went wrong.

Tags: lp-bugs
Revision history for this message
William Grant (wgrant) wrote :

In fact, it's worse than that - the Subscribe link gets partly AJAXified, to the point that it doesn't do anything at all.

This is really bad, as it means users can't unsubscribe from the bugs which they are most likely to want to unsubscribe from - the massive ones with hundreds of duplicates and excessive numbers of 'unsubscribe' comments.

summary: - Subscribers portlet main content silently fails to load
+ Subscribers portlet doesn't handle failure to load list well at all
Revision history for this message
Deryck Hodge (deryck) wrote :

Agreed this is a pain. And also concerned that the subscribe link is hooked up via JavaScript if the subscribers portlet doesn't load. The js hooks are only supposed to be established after a custom portlet loaded event.

In the meantime, appending +subscribe to a bug page URL should offer a way to unsubscribe until this is fixed.

Changed in malone:
importance: Undecided → High
milestone: none → 2.2.6
status: New → Triaged
assignee: nobody → Deryck Hodge (deryck)
Revision history for this message
Eleanor Berger (intellectronica) wrote :

If there are frequent timeouts, we should fix those, not build lots of error handling around them. Having said that, we don't really display anything for browsers with no JS, so we should consider fixing that.

Revision history for this message
Gavin Panella (allenap) wrote :

For more than just this problem, it would be good to know,
server-side, if the client has Javascript turned on. Then we know
whether to render the subscriber portlet inline or if we can defer it.

Deryck Hodge (deryck)
Changed in malone:
status: Triaged → In Progress
Revision history for this message
Diogo Matsubara (matsubara) wrote : Bug fixed by a commit

Fixed in devel r8694.

Changed in malone:
status: In Progress → Fix Committed
Deryck Hodge (deryck)
Changed in malone:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.