Skip to content

refactor: lndrest get_payment_status and pay_invoice use api v2#3470

Merged
dni merged 9 commits intodevfrom
fix/lndrest
Nov 10, 2025
Merged

refactor: lndrest get_payment_status and pay_invoice use api v2#3470
dni merged 9 commits intodevfrom
fix/lndrest

Conversation

@dni
Copy link
Copy Markdown
Member

@dni dni commented Nov 3, 2025

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 3, 2025

Codecov Report

❌ Patch coverage is 78.72340% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.42%. Comparing base (b4eccb9) to head (4692343).
⚠️ Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
lnbits/wallets/lndrest.py 78.72% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #3470      +/-   ##
==========================================
- Coverage   56.10%   54.42%   -1.68%     
==========================================
  Files         117      117              
  Lines       15688    15704      +16     
==========================================
- Hits         8801     8547     -254     
- Misses       6887     7157     +270     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dni dni requested a review from motorina0 November 3, 2025 08:58
@dni dni marked this pull request as draft November 3, 2025 10:23
@dni dni marked this pull request as ready for review November 3, 2025 11:16
@dni dni added this to the 1.3.2 milestone Nov 3, 2025
Base automatically changed from fix/lndgrpc to dev November 10, 2025 14:07
dni and others added 9 commits November 10, 2025 15:09
…iceV2

more verbose logging of status

typo

add update grpc script

update grpc files

remove ignores

fix imports

imports and enums

clean clean clean

rename fixtures

fixtures add fees

fix calls

more fuixtures

fixup type

dumb error :D

remove inflight update
@dni dni merged commit e038ceb into dev Nov 10, 2025
44 checks passed
@dni dni deleted the fix/lndrest branch November 10, 2025 15:26
dni added a commit that referenced this pull request Nov 10, 2025
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.

2 participants