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

Skip to content

Conversation

@arbadacarbaYK
Copy link
Contributor

@arbadacarbaYK arbadacarbaYK commented Apr 1, 2024

Common misconceptioin is that the top up is related to the funds on the funding source.

Success msg extended with info that correlated funds on funding source are needed and the amount is virtual until fitting.

@dni dni requested a review from talvasconcelos April 5, 2024 05:15
@dni
Copy link
Member

dni commented Apr 8, 2024

i think the explanation should be in the frontend not the API. i could also be translated there.

Common misconceptioin is that the top up related to the funds on the funding source. 

Success msg  extended with info that correlated funds on funding source are needed and the amount is virtual until fitting.
@codecov
Copy link

codecov bot commented Apr 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.76%. Comparing base (d78f6a1) to head (c99c237).
Report is 2 commits behind head on dev.

❗ Current head c99c237 differs from pull request most recent head eec184f. Consider uploading reports for the commit eec184f to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2381      +/-   ##
==========================================
- Coverage   60.79%   60.76%   -0.03%     
==========================================
  Files          65       65              
  Lines        9336     9337       +1     
==========================================
- Hits         5676     5674       -2     
- Misses       3660     3663       +3     

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

@motorina0
Copy link
Collaborator

Fixed:
image

Note: i18n does not work on admin.js (yet).

Copy link
Member

@dni dni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! that small wording change actually improved the code in the end :)

@dni dni requested a review from motorina0 April 17, 2024 07:50
@dni dni merged commit d9880c4 into lnbits:dev Apr 17, 2024
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