-
Notifications
You must be signed in to change notification settings - Fork 192
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
fix(admin-tool): All Forms dropdown on the Donation Export only uses first ten forms #3303 #3346
Conversation
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.
@raftaar1191 can you confirm that form dropdown ajax supported or not
@ravinderk This PR is ready for review and merge |
Yes @ravinderk it support ajax form Search too |
@@ -189,6 +189,10 @@ function give_export_donation_form_search_args( $args ) { | |||
$fields = isset( $_POST['fields'] ) ? $_POST['fields'] : null; | |||
parse_str( $fields ); | |||
|
|||
if ( ! empty( $give_forms_categories ) || ! empty( $give_forms_tags ) ) { |
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.
@raftaar1191 if ajax enabled for form for category and tag then why we have -1
here
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.
@raftaar1191 everything is good here
@@ -189,6 +189,10 @@ function give_export_donation_form_search_args( $args ) { | |||
$fields = isset( $_POST['fields'] ) ? $_POST['fields'] : null; | |||
parse_str( $fields ); | |||
|
|||
if ( ! empty( $give_forms_categories ) || ! empty( $give_forms_tags ) ) { |
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.
@raftaar1191 everything is good here
Description
PR to fix #3303
How Has This Been Tested?
Tested by exporting donation by selecting the cat and tag and then exporting the donation
Video Link: https://screencast-o-matic.com/watch/cF1l1eFfAa
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist: