Closed Bug 1098629 Opened 5 years ago Closed 5 years ago
Support the retry button on the error bar in more cases
38 bytes, text/x-review-board-request
There are some more cases where we can add retry support which weren't added in bug 1074670. There are also some cases (e.g. network errors) where the button is hooked up but not shown because the 2nd bar isn't shown due to the lack of a details message. We can default to the following strings in those cases: generic_failure_no_reason2=Would you like to try again? retry_button=Retry
/r/609 - Bug 1098629 - WIP to support the retry button on the error bar in more cases. Pull down this commit: hg pull review -r 1a7a478e5eb7ecf124b61ed0332f0ae5b83d1844
Moving this to P2 based on our new priority definitions.
Priority: P1 → P2
moving to 38 based on time - can consider uplifting if done early/hits bar/low risk.
moving to 38 based on time (though will definitely land/uplift if it lands in 37.3)- can consider uplifting if done early/hits bar/low risk. if this lands earlier - definitely want to uplift. especially since the retry button can work now that some of Pkerrs patches are landing for push retries when server is down.
hi matt, you have a patch - will you be putting up for review in 38.1?
To be tracked by Hello Team.
Iteration: 38.1 - 26 Jan → ---
5 years ago
Attachment #8522571 - Flags: review?(pkerr)
/r/609 - Bug 1098629 - Support the retry button on the error bar in more cases. r=pkerr Pull down this commit: hg pull review -r 7e3bd82f65dc0254851ede1bac5ed7277c0c31df
This still seems to work but I don't remember context of what else I was planning on doing in this bug. I would appreciate a thorough review as I have forgotten some context.
https://reviewboard.mozilla.org/r/607/#review1645 ::: browser/components/loop/MozLoopService.jsm (Diff revision 2) > + log.trace(); It seemed useful to always log the stack trace of when setError was called. This only logs when the LogLevel includes Trace.
Flags: in-testsuite? → in-testsuite+
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
This seems to be covered to some extent by automated tests. Is manual testing also needed here?
QA Contact: anthony.s.hughes
It would be good but it's probably hard to replicate various server/network issues that this affects.
It looks like this didn't get uplifted since I stopped following Hello development. Shall I request uplift? It adds a "Retry" button on panel error bars in more cases which helps with intermittent network/server issues or when a session expires.
I don't see a strong reason to uplift this, maybe RT feels differently.
You need to log in before you can comment on or make changes to this bug.