Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Mobile
Minds Mobile
  • Project overview
  • Repository
  • Issues 165
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 15
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Commits
  • Issue Boards
  • Minds
  • Minds MobileMinds Mobile
  • Issues
  • #1622

Closed
Open
Opened 2 weeks ago by Bill Ottman@ottman:dragon_face:0 of 2 tasks completed0/2 tasks
Report abuse New issue

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

  1. Lose connectivity.
  2. 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

Screenshot_20191214-132038_Minds

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"

Edited 2 weeks ago by Brian Hatchet

Related issues
0

Related merge requests
1
  • Remote action with retry and different messages for network, offline and generic errors.
    !471
    Mobile 3.15
  • Discussion 6
  • Designs 0
  • Bill Ottman :dragon_face: @ottman added Platform::Mobile::Android Priority::2 - Normal Product::Blogs Status::Backlog Type::Bug scoped labels 2 weeks ago

    added scoped labels

  • Bill Ottman
    Bill Ottman :dragon_face: @ottman · 2 weeks ago
    Owner

    Was trying to vote

  • Ben Hayward @benhayward.ben added Status::Scheduling scoped label and automatically removed Status::Backlog label 2 weeks ago

    added scoped label and automatically removed label

  • Ben Hayward @benhayward.ben added Status::Validation scoped label and automatically removed Status::Scheduling label 2 weeks ago

    added scoped label and automatically removed label

    • Ben Hayward
      Ben Hayward @benhayward.ben · 2 weeks ago
      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?

    • Collapse replies
    • Juan Manuel Solaro
      Juan Manuel Solaro @juanmsolaro · 2 weeks ago
      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.

    • Ben Hayward
      Ben Hayward @benhayward.ben · 2 weeks ago
      Developer

      Sounds good, thank you for your quick response @juanmsolaro

    • Martin Santangelo
      Martin Santangelo @msantang78 · 2 weeks ago
      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.

    • Please register or sign in to reply
  • Ben Hayward @benhayward.ben changed title from Unuseful error message to Improve mobile error message and add retry logic 2 weeks ago

    changed title from to

  • Ben Hayward @benhayward.ben changed the description 2 weeks ago

    changed the description

  • Ben Hayward @benhayward.ben added Status::Scheduling scoped label and automatically removed Status::Validation label 2 weeks ago

    added scoped label and automatically removed label

  • Brian Hatchet :speech_balloon: @brianhatchet changed the description 2 weeks ago

    changed the description

  • Brian Hatchet :speech_balloon: @brianhatchet changed time estimate to 4h 2 weeks ago

    changed time estimate to 4h

  • Brian Hatchet :speech_balloon: @brianhatchet changed weight to 4 2 weeks ago

    changed weight to 4

  • Brian Hatchet :speech_balloon: @brianhatchet added Status::Backlog scoped label and automatically removed Status::Scheduling label 2 weeks ago

    added scoped label and automatically removed label

  • Brian Hatchet :speech_balloon: @brianhatchet added Squad::Green scoped label 2 weeks ago

    added scoped label

  • Brian Hatchet :speech_balloon: @brianhatchet changed milestone to %Mobile 3.15 2 weeks ago

    changed milestone to %Mobile 3.15

  • Brian Hatchet :speech_balloon: @brianhatchet added 1 deleted label 2 weeks ago

    added 1 deleted label

  • Brian Hatchet :speech_balloon: @brianhatchet assigned to @msantang78 2 weeks ago

    assigned to @msantang78

  • John Ottman
    John Ottman @Johnthetester · 2 weeks ago
    Reporter

    This error happens when a group post appears in the mobile newsfeed and user votes on it

  • Martin Santangelo @msantang78 removed 1 deleted label 1 week ago

    removed 1 deleted label

  • Martin Santangelo @msantang78 added Sprint::12/25 Wan Wren scoped label 1 week ago

    added scoped label

  • Martin Santangelo @msantang78 added Status::InProgress scoped label and automatically removed Status::Backlog label 1 week ago

    added scoped label and automatically removed label

  • Martin Santangelo @msantang78 mentioned in merge request !471 (merged) 1 week ago

    mentioned in merge request !471 (merged)

  • Martin Santangelo @msantang78 added Status::BuddyReview scoped label and automatically removed Status::InProgress label 1 week ago

    added scoped label and automatically removed label

  • Martin Santangelo @msantang78 assigned to @juanmsolaro 1 week ago

    assigned to @juanmsolaro

  • Brian Hatchet :speech_balloon: @brianhatchet added Sprint::01/01 - Xerothermic Xenarthra scoped label and automatically removed Sprint::12/25 Wan Wren label 1 day ago

    added scoped label and automatically removed label

  • Juan Manuel Solaro @juanmsolaro added Status::Review scoped label and automatically removed Status::BuddyReview label 5 hours ago

    added scoped label and automatically removed label

  • Brian Hatchet :speech_balloon: @brianhatchet closed 1 hour ago

    closed

  • Brian Hatchet :speech_balloon: @brianhatchet removed Status::Review label 1 hour ago

    removed label

Please register or sign in to reply
2 Assignees
Martin Santangelo's avatar
Juan Manuel Solaro's avatar
none
Epic
None
Mobile 3.15
Milestone
Mobile 3.15
Time tracking
Estimated: 4h
None
Due date
None
6
Labels
Platform::Mobile::Android Priority::2 - Normal Product::Blogs Sprint::01/01 - Xerothermic Xenarthra Squad::Green Type::Bug
4
Weight
4
Confidentiality
Not confidential
Lock issue
Unlocked
6
6 participants
user avatar
Martin Santangelo
user avatar
Juan Manuel Solaro
user avatar
Brian Hatchet
user avatar
John Ottman
user avatar
Ben Hayward
user avatar
Bill Ottman
Reference: minds/mobile-native#1622