-
-
Couldn't load subscription status.
- Fork 371
Clarified top-up success msg #2381
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
Conversation
|
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 ReportAll modified and coverable lines are covered by tests ✅
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. |
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.
thanks! that small wording change actually improved the code in the end :)
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.