-
Notifications
You must be signed in to change notification settings - Fork 191
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
Issues/2397 #2399
Issues/2397 #2399
Conversation
@@ -419,177 +419,29 @@ function give_undo_donation( $payment_id ) { | |||
* @return object $stats Contains the number of payments per payment status. | |||
*/ | |||
function give_count_payments( $args = 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.
@DevinWalker I update this function and removed some unused filter also, let me know what do you think
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.
Let me know if you want to add backward compatibility for give_count_payments_where
and give_count_payments_join
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.
Looks good @ravinderk
@ravinderk have a Question for you We can order table with donation from id, date etc but what I see is that I am not able to use any of the filter Is it normal? |
@DevinWalker and @mehul0810 can you confirm sorting is working fine for you |
@ravinderk sorting what? |
@DevinWalker Sorting on donation listing page with filters |
@ravinderk looks like sorting is working but paging is broken https://goo.gl/qUqutJ Can you fix that ASAP? |
Description
THIS PR WILL RESOLVE #2397
How Has This Been Tested?
Manually apply gateway sorting with form_id
Screenshots (jpeg or gifs if applicable):
Types of changes
New feature (non-breaking change which adds functionality)
Checklist: