-
-
Notifications
You must be signed in to change notification settings - Fork 326
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
alby: fix balance reporting and error handling + opennode: fix error handling #2221
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #2221 +/- ##
==========================================
- Coverage 59.23% 59.21% -0.02%
==========================================
Files 60 60
Lines 8991 8994 +3
==========================================
Hits 5326 5326
- Misses 3665 3668 +3 ☔ View full report in Codecov by Sentry. |
Let's not postpone to another milestone. The Alby integration is now unusable so it does not make sense to keep it broken for 0.12.0. Can we have another review @motorina0 @arcbtc plz? |
|
||
return StatusResponse(None, data) | ||
data = r.json() | ||
assert data["unit"] == "sat" |
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.
nitpick:
assert data["unit"] == "sat" | |
assert data["unit"] == "sat", f"Expected unit 'sat', but was '{data["unit"]}'" |
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.
utACK
cc @bitkarrot for testing