-
Notifications
You must be signed in to change notification settings - Fork 25
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
Update the cashu token QR #32
Update the cashu token QR #32
Conversation
closes #26
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## dev #32 +/- ##
==========================================
- Coverage 48.37% 48.23% -0.15%
==========================================
Files 25 25
Lines 1017 1020 +3
Branches 226 228 +2
==========================================
Hits 492 492
- Misses 461 463 +2
- Partials 64 65 +1
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but there is one more thing to change before closing this issue.
- claiming the token after scanning (Maybe needs to check if the scanned value includes the cashu prefix or not)
Line 113 in 0cfef2e
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥜
closes #26