Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better error handling of transactions being not send #15464

Closed
STew790 opened this issue Nov 20, 2024 · 6 comments · Fixed by #15781
Closed

Better error handling of transactions being not send #15464

STew790 opened this issue Nov 20, 2024 · 6 comments · Fixed by #15781
Labels
bug Something isn't working as expected mobile Suite Lite issues and PRs QA NOK Issue doesn't pass the QA and some changes are needed.

Comments

@STew790
Copy link
Contributor

STew790 commented Nov 20, 2024

Describe the bug
There is a chance that transactions might not be sent and there are no errors that would inform the user about what's happening. For example backends might not be reachable and there is "forever" loading when you click on the final "Send" button.

Info:

  • Mobile Suite Version: version/commit
  • OS: name, version

Expected behavior
I should be better informed what is happening and why is the transaction not sending

@STew790 STew790 added bug Something isn't working as expected mobile Suite Lite issues and PRs labels Nov 20, 2024
@github-project-automation github-project-automation bot moved this to 🎯 To do in Suite Mobile Nov 20, 2024
@PeKne PeKne self-assigned this Nov 25, 2024
@PeKne PeKne moved this from 🎯 To do to 🤝 Needs QA in Suite Mobile Dec 10, 2024
@PeKne
Copy link
Contributor

PeKne commented Dec 10, 2024

notes for QA:

  • if the transaction fails to send there should be always dispalyed and modal informing user that something went wrong.

@STew790
Copy link
Contributor Author

STew790 commented Dec 11, 2024

QA NOK
I just got a forever "sending" on SOL and it didn't show any error for about 3-5 mins so I manually cancelled it. There was no failed transaction, it just timed out probably.
Whats weird is that after like another 10mins I unlocked my phone and saw this:
image

@STew790 STew790 moved this from 🤝 Needs QA to 🎯 To do in Suite Mobile Dec 11, 2024
@STew790 STew790 added the QA NOK Issue doesn't pass the QA and some changes are needed. label Dec 11, 2024
@PeKne
Copy link
Contributor

PeKne commented Dec 11, 2024

@vytick Can you look at it please? I suspect that there will be some problem in communication with SOL backend or connect.

@vytick
Copy link
Contributor

vytick commented Dec 12, 2024

I'll check, if there is anything I can find.

@vytick
Copy link
Contributor

vytick commented Dec 13, 2024

I haven't found anything nor i was able to reproduce.

@PeKne PeKne unassigned PeKne and shenkys Dec 16, 2024
@matejkriz matejkriz reopened this Dec 16, 2024
@github-project-automation github-project-automation bot moved this from 🎯 To do to 🏃‍♀️ In progress in Suite Mobile Dec 16, 2024
@matejkriz matejkriz moved this from 🏃‍♀️ In progress to 🎯 To do in Suite Mobile Dec 16, 2024
@matejkriz
Copy link
Member

Closing this stale issue, feel free to create a new one (or reopen this) if anyone report it again.

@matejkriz matejkriz moved this from 🎯 To do to ✅ Approved in Suite Mobile Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as expected mobile Suite Lite issues and PRs QA NOK Issue doesn't pass the QA and some changes are needed.
Projects
Status: ✅ Approved
Development

Successfully merging a pull request may close this issue.

5 participants