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

Removed give_export_all_customers() and code cleanup #1595

Merged
merged 10 commits into from
Mar 30, 2017

Conversation

DevinWalker
Copy link
Member

Description

Addresses #1571

  • Removed give_export_all_customers()
  • Removed other unused variables and fields
  • PHPdocs improments

Types of changes

  • Code improvements

Checklist:

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

Devin Walker added 10 commits March 9, 2017 11:57
* release/1.8.6:
  Improved payment status language
  Reordered to fix travis error
  Attempting to fix broken PHPunit tests
  Fix: typo in currency section setting
  Fix: generate level class and id with price_id instead of
  fix #1592
  Update README.md
  Update Documentation.
  New POT file for 1.8.5
  Change log updated with completed issues
  New option to toggle billing details and corresponding logic #1573
  Version bumped, change log added - more to add
  Code formatting
  Added exception for offline donations meta_key upgrades #1579
  Ensure upgrade can run on any WP-admin page, don’t rely on give_vars which was throwing a console error and the confirm window would never show #1580
  Updated give_logged_in_only to return enabled if metakey `_give_logged_in_only` returns empty #1578
  Fix typo
  Add comment
  Render continue button title setting field conditionaly in shortcode generator popup
  Show/Hide on continue button title in widget form
@DevinWalker DevinWalker merged commit 170e3e1 into release/1.8.6 Mar 30, 2017
@DevinWalker DevinWalker deleted the issue/1571 branch April 27, 2017 18:01
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.

1 participant