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

Skip to content

Conversation

@dni
Copy link
Member

@dni dni commented Mar 31, 2025

  • if phoenixd return 404 on incoming payments the invoice was either never created or is expired
  • if phoenixd return 404 on outgoing they were never created

@codecov
Copy link

codecov bot commented Mar 31, 2025

Codecov Report

Attention: Patch coverage is 78.72340% with 10 lines in your changes missing coverage. Please review.

Project coverage is 61.01%. Comparing base (0bbb0ac) to head (69c29e8).
Report is 2 commits behind head on dev.

Files with missing lines Patch % Lines
lnbits/wallets/phoenixd.py 78.72% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #3075      +/-   ##
==========================================
- Coverage   61.04%   61.01%   -0.03%     
==========================================
  Files         110      110              
  Lines       14320    14343      +23     
==========================================
+ Hits         8741     8752      +11     
- Misses       5579     5591      +12     

☔ 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 force-pushed the fix/incoming-payment-fees branch from f14219b to a323011 Compare April 2, 2025 11:23
@dni dni force-pushed the fix/incoming-payment-fees branch from 71d56dd to b91cee0 Compare May 5, 2025 09:02
@dni dni marked this pull request as draft May 5, 2025 09:04
@dni dni closed this May 12, 2025
@blackcoffeexbt
Copy link
Collaborator

@motorina0 this would be a useful PR to deal with phoenixd with many pending invoices. Can we reopen the PR? For example, I have an instance with nearly 4000 pending invoices.

@motorina0 motorina0 reopened this May 28, 2025
@motorina0 motorina0 force-pushed the fix/incoming-payment-fees branch from b91cee0 to 451bd19 Compare May 28, 2025 09:09
@motorina0 motorina0 self-requested a review July 1, 2025 10:59
@motorina0 motorina0 assigned motorina0 and unassigned dni Jul 1, 2025
@motorina0 motorina0 force-pushed the fix/incoming-payment-fees branch from 451bd19 to 63e23eb Compare July 3, 2025 07:26
dni added 5 commits July 3, 2025 12:07
- if phoenixd return 404 on incoming payments the invoice was either
never created or is expired
- if phoenixd return 404 on outgoing they were never created
@motorina0 motorina0 force-pushed the fix/incoming-payment-fees branch from 63e23eb to e081fbd Compare July 3, 2025 09:08
@motorina0 motorina0 marked this pull request as ready for review July 4, 2025 08:41
@motorina0 motorina0 merged commit bf8b33a into dev Jul 4, 2025
43 checks passed
@motorina0 motorina0 deleted the fix/incoming-payment-fees branch July 4, 2025 09:01
dni added a commit that referenced this pull request Jul 15, 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.

3 participants