Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@motorina0
Copy link
Collaborator

@motorina0 motorina0 commented Feb 26, 2024

Required while testing the APIs, but can be useful in general.

The new endpoint uses check_user_exists access level, so full user permissions is required to get all wallets.

@motorina0 motorina0 marked this pull request as ready for review February 26, 2024 15:11
@codecov
Copy link

codecov bot commented Feb 26, 2024

Codecov Report

Attention: Patch coverage is 85.71429% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 58.53%. Comparing base (54c6faa) to head (5b02b56).
Report is 14 commits behind head on dev.

Files Patch % Lines
lnbits/core/views/api.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2299      +/-   ##
==========================================
- Coverage   59.17%   58.53%   -0.64%     
==========================================
  Files          60       61       +1     
  Lines        9036     9213     +177     
==========================================
+ Hits         5347     5393      +46     
- Misses       3689     3820     +131     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@motorina0 motorina0 merged commit e8aa498 into dev Feb 27, 2024
@motorina0 motorina0 deleted the feat_all_user_wallets_api branch February 27, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants