Skip to content
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(shortcode): select donation form alert must be open using modal api #3244 #3282 #3286

Closed
wants to merge 143 commits into from

Conversation

Sidsector9
Copy link
Contributor

Closes #3282

Description

This PR replaces the native alert dialog with GiveModalWarning.

How Has This Been Tested?

By adding a form in a shortcode without choosing a form.

Screenshots (jpeg or gifs if applicable):

warning

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows has proper inline documentation.

mehul0810 and others added 30 commits April 20, 2018 17:13
fix(form): do not unformat the min/max amount using JS functions.
fix(admin-form): prevent js notice when edit price type setting field #3161
fix(admin-setting): prevent setting menu to appear on top of modal #3164
* release/2.1.1:
  fix(admin-shortcode): clarify Donation Form Grid shortcode generator
  fix(db-upgrade): do not show upgrade notice on WP update page
  fix(donation): update default donation amount
  fix(donation): update form max value
  fix(donation): improve if condition
* release/2.1.2:
  doc(release): Add change log for 2.1.1 and run npm prod for new POT file
DevinWalker and others added 26 commits May 23, 2018 08:01
* release/2.1.3:
  style(code): improve code formatting
  fix(admin-settings): Disables radio button if payment gateway is disabled
  fix(donation): improved the logic of checking min/max amount.
  feat(admin-email): Add email header for 3 email notification types Added email header option for new donation, receipt and offline donation instructions.
  feat(admin-email): Add setting field for email header
* release/2.1.4:
  doc(release): update new branch version
  doc(release): add change log for 2.1.3 and run npm prod
  feat(admin-tool): Change action and filter name
  feat(admin-tool): Added a new action and filter
  doc(release): bump version number
  doc(release): run npm production before release and add change log
* release/2.1.4:
  doc(release): gen new POT file for release
  fix: improve error description
  fix(donation): update code to support latin characters for donations
  doc(release): new POT generated for master branch since we merged commits pre-merge to master
fix(compat): prevent missing donation data with Restrict Content Pro #2710
# Conflicts:
#	includes/admin/donors/donor-actions.php
#	includes/admin/payments/view-payment-details.php
#	includes/misc-functions.php
fix(admin-tool): export and Import of Donations must be in sync without errors #2135
@ravinderk
Copy link
Collaborator

@Sidsector9 recreate this pr as we discussed.

@ravinderk ravinderk closed this May 30, 2018
ravinderk added a commit that referenced this pull request May 30, 2018
fix(shortcode): select donation form alert must be open using modal api #3244 #3282 #3286
@Sidsector9 Sidsector9 deleted the issue-3282 branch May 30, 2018 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants