-
Notifications
You must be signed in to change notification settings - Fork 199
Feature: add donation overview page #7985
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
Feature: add donation overview page #7985
Conversation
…donationDetails array
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.
Great job! I’m approving this 👏
There are just a couple of small things we might want to revisit later:
- I’m wondering if it wouldn’t be better for gateways to provide their own view URL, maybe as a method in the Gateway API. That way, given a
transactionId
, each gateway could return its formatted URL. The current approach might make it harder for other gateways (including 3rd-party ones) to have their view URLs shown in the Donation Details page. - Also, we’re not showing the gateway title properly at the moment. Maybe we could get the title directly from the gateway and include it already formatted in the API response?
Nothing blocking for now, just some ideas to consider moving forward 🙂
@pauloiankoski Thanks! For the Payment method title - I actually did update where we were pulling that information from but did not update the visuals for you in time. Its updated now but I am however working with Jeffrey to get icons for all the gateways. I'll make sure to circle back on how we are providing the urls! Could be a good Fun-Friday thing I might bring you in on 😄 |
e4c7af1
into
epic/donation-details-admin-page
@JoshuaHungDinh @pauloiankoski we should definitely not be hardcoding gateway links here. There is an existing filter that gateways use to display their link on the current details page that we can use for now since all gateways are already using it: apply_filters( "give_payment_details_transaction_id-{$gatewayId}", $gatewayTransactionId, $donationId ) We can create a dedicated method in the gateway api for this as well - but should start by using the filter |
Resolves GIVE-2601
Description
This PR introduces a new Donation Overview page to the GiveWP admin, providing a comprehensive summary and key insights for individual donations. The overview page is designed to improve the admin experience by consolidating donation details, statistics, and related actions into a single, easily navigable interface.
Changes to note:
src/admin/components
Component logic
DonationStats
DonationSummaryGrid
Affects
Donation Overview page & components
Visuals
Testing Instructions
General
Component: DonationStats
If the donation includes event ticket sales:
If the donation does not include event ticket sales:
If the Fee Recovery add-on is not installed:
If the Fee Recovery add-on is installed:
Component: DonationSummaryGrid
If gatewayViewUrl is available:
Pre-review Checklist
@unreleased
tags included in DocBlocks