-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Always show charge and authorised amount in payment section of order details #4821
Conversation
🦋 Changeset detectedLatest commit: 091c2bf The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
If the payment section now will show all payments info, aggregated from all transactions in a given order, we should inform the user about that:
- We can call the section "Payments summary" instead of "Payment"
- We can add small text right below the title that explains what it presents eg. "A summary of all payments from registered transactions"
What do you think?
…ashboard into merx-342-refund-payments
…details (#4821) * Temp solution * Show charge and autorized amount * Add changeset * Restore messages * Improve showing amount * Improve copy --------- Co-authored-by: Michał Droń <droniu@droniu.dev>
AC:
What type of PR is this?
Related Issues or Documents
Usage Instructions, Screenshots, Recordings
Have you written tests?
[Optional] Description