Improve mobile error message and add retry logic
Summary
The error message shown on a failed request (as below on failed upvote) is not helpful to the user or developer.
Steps to reproduce
- Lose connectivity.
- Try upvote something.
Platform information
Cross-platform mobile.
App version
v3.12.0
What is the current bug behavior?
Generic error is shown to user.
What is the expected correct behavior?
should show an alert / modal to the user informing them of the error (voting could not be done). Also a retry button would be ideal.
Relevant logs and/or screenshots
Possible fixes
- Add specific error messages to retry failures
- Add a retry button for failed network requests (if possible without significant re-architecture)
/label ~"T - Bug" ~"S - Triage:new"
added scoped labels
- Owner
Was trying to vote
added scoped label and automatically removed label
added scoped label and automatically removed label
- Developer
Putting this into validation whilst we check that there is something that can be done.
@msantang78 @brianhatchet @juanmsolaro, is there anything that we can do in this area to make this error more verbose and relevant to the crash?
- Developer
If i remember correctly we talked about this last week. For what we saw, the message it´s not related to a crash on the app side but a fail on the request. This can be connectivity issues. We need to show some kind of alert to the user, since the voting could not be done. Changing the message is easy i think, we have to create one specific for this. But we also talked, if i'm not mistaken, about a retry button.
- Developer
Sounds good, thank you for your quick response @juanmsolaro
- Developer
Yeah, that error usually is shown when the call to the server fails and a retry will improve a lot the UX. We should implement a generic retry logic to use in all the actions, it should show an error message to the user with the retry button that repeats the same call to the server. Also, we can add a message if the connectivity service detect that the user is offline.
changed title from to
changed the description
added scoped label and automatically removed label
changed the description
changed time estimate to 4h
changed weight to 4
added scoped label and automatically removed label
added scoped label
changed milestone to %Mobile 3.15
added 1 deleted label
assigned to @msantang78
- Reporter
This error happens when a group post appears in the mobile newsfeed and user votes on it
removed 1 deleted label
added scoped label
added scoped label and automatically removed label
mentioned in merge request !471 (merged)
added scoped label and automatically removed label
assigned to @juanmsolaro
added scoped label and automatically removed label
added scoped label and automatically removed label
closed
removed label