-
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
feat(admin-tool): merge CSV toolbox into Give core #2875 #2975
Conversation
@raftaar1191 Arrange checkboxes in columns form like below image: |
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.
Check my comment: #2975 (comment)
var give_form_id; | ||
|
||
// Check for form ID. | ||
if ( ! ( |
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 improve code formatting here.
output_give_donations_fields( response ); | ||
|
||
} else { | ||
alert( 'An AJAX error occurred.' ); |
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 this must be translatable
@ravinderk updated the PR as you have mentioned in requested changes |
/** | ||
* Ajax call to get donation fields. | ||
*/ | ||
$( '.give-export_donations #give-export_donations-form #give_form_for_csv_export' ).chosen().change( function () { |
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 please remove this from create a new JS file called admin-exports.js
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.
Okay @DevinWalker doing that
@DevinWalker Update PR with all the suggestion that you have made |
@raftaar1191 can you fix the conflict in this PR? |
# Conflicts: # includes/admin/tools/views/html-admin-page-exports.php
@DevinWalker merge conflict resolved. |
Description
PR to fix #2875
How Has This Been Tested?
Manually Tested after removing the Export Donation History Code and then add the CSV ToolBox Code to Give Core 2.1 and exporting the donation from the CSV
Task List
Screenshots (jpeg or gifs if applicable):
View the Export Page
With out selecting the Donation Form:
When selecting the Donation Form:
Types of changes