-
Notifications
You must be signed in to change notification settings - Fork 107
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
feat: [IOBP-782] Add biz events receipt migration banner info #6188
feat: [IOBP-782] Add biz events receipt migration banner info #6188
Conversation
…-add-biz-events-migration-banner-info
…vents-new-definitions
…-add-biz-events-migration-banner-info
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6188 +/- ##
==========================================
- Coverage 48.42% 46.98% -1.44%
==========================================
Files 1488 1802 +314
Lines 31617 36546 +4929
Branches 7669 8649 +980
==========================================
+ Hits 15311 17172 +1861
- Misses 16238 19317 +3079
+ Partials 68 57 -11
... and 1418 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
…vents-new-definitions
…-add-biz-events-migration-banner-info
🎉 All dependencies have been resolved ! |
…ents-migration-banner-info
…n-banner-info' into IOBP-782-add-biz-events-migration-banner-info
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.
Wrong locale configuration for the English language
Short description
This PR adds a banner inside the transaction details to inform the user if that transaction doesn't have the PDF available because it has been imported from the PM (the old payment gateway);
List of changes proposed in this pull request
PM
(that means this transaction is imported);Get the receipt
button inside the transaction details if the origin isPM
How to test
With the dev server, try to edit the origin of a transaction setting the
origin
property toPM
, you should be able to see the banner info and there must not be visible the CTA to download the receipt;Preview
preview.mov