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

Go back from Invoice screen without checking payment #165

Closed
KKA11010 opened this issue Sep 8, 2023 · 0 comments · Fixed by #210
Closed

Go back from Invoice screen without checking payment #165

KKA11010 opened this issue Sep 8, 2023 · 0 comments · Fixed by #210
Assignees
Labels
bug Something isn't working

Comments

@KKA11010
Copy link
Collaborator

KKA11010 commented Sep 8, 2023

Describe the bug
When i transfer sats to the mint but after paying the invoice i go back in the menu instead of claiming the nuts it first doesn't show the nuts in my balance. This is not a real issue as the balance gets updated if i wait long enough or when i deposit again so no issue there. However the transaction is not registered in my transaction history when i cut this proces of. Maybe nice to put a disclaimer in the collect nuts section that if you dont collect the nuts it might take time to arrive in the wallet and the transaction wont show up in the transaction history. Its not a big problem as the nuts do arrive.

Expected behavior
Invoice payment check should be automated, avoid the user to press "check payment" manually-

suggestion
check if invoice is payed while going back from the invoice screen and handle the payment. also provide automated payment check and remove the check payment button

@KKA11010 KKA11010 added the bug Something isn't working label Sep 8, 2023
@KKA11010 KKA11010 self-assigned this Sep 8, 2023
KKA11010 added a commit that referenced this issue Sep 13, 2023
* auto-check LN payment in invoice screen Fixes #165

* update

* update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant