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.

Sign up for GitHub

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

Jump to bottom

Stop showing error messages of inactive connected wallets #252

Open
MoritzKa opened this issue Jan 18, 2025 · 1 comment · May be fixed by #254
Open

Stop showing error messages of inactive connected wallets #252

MoritzKa opened this issue Jan 18, 2025 · 1 comment · May be fixed by #254
Assignees
@im-adithya

Comments

@MoritzKa
Copy link

@stackingsaunter reported that he sees error messages in Alby Go of connected wallets that are not active /not selected

Managed to capture the error after opening the app, when selected wallet was 100% online and worked.
I paid for groceries 5s later
I had 3 wallets then and one was closed. So i suppose it was it'd error.

Possible solution: the error should be displayed only when it's related to selected wallet OR the error should mention which wallet cannot be reached

Image

Possible solution: the error should be displayed only when it's related to selected wallet OR the error should mention which wallet cannot be reached.
@rolznz
Copy link
Contributor

rolznz commented Jan 18, 2025

I wonder if this is an issue with some pending NWC command that is stalled when the app is backgrounded, and then once the app is focused again, it triggers this error due to the inactivity?

@getAlby getAlby deleted a comment Jan 18, 2025
@getAlby getAlby deleted a comment from Fullstack558 Jan 18, 2025
@im-adithya im-adithya linked a pull request Jan 21, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@im-adithya im-adithya

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: check wallet id before throwing error getAlby/go
3 participants
@MoritzKa @rolznz @im-adithya