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

Feat/pricing fields #293

Merged
merged 20 commits into from
Aug 30, 2022
Merged

Conversation

justlevine
Copy link
Member

@justlevine justlevine commented Aug 22, 2022

What

Adds support for pricing fields (product, quantity, option, shipping, total) to the schema.

Why

#195

How

  • feat: add support for Product, Quantity, Option, Shipping and Total GF fields.
  • test: ensure $_gf_state is reset between tests.
  • fix: ensure up-to-date entry is used to prepare field values for save.

Testing Instructions

Additional Info

Checklist:

  • My code is tested to the best of my abilities.
  • My code follows the WordPress Coding Standards.
  • My code has proper inline documentation.
  • I have added unit tests to verify the code works as intended.

@justlevine justlevine force-pushed the feat/pricing-fields branch from dba1bff to 253e604 Compare August 28, 2022 17:57
@justlevine justlevine added the safe to test ✔ This PR can be run using repository secrets label Aug 28, 2022
@justlevine justlevine force-pushed the feat/pricing-fields branch from 8b3936b to 222f359 Compare August 28, 2022 18:13
@justlevine justlevine force-pushed the feat/pricing-fields branch 2 times, most recently from 62d6c3e to 52bb138 Compare August 29, 2022 10:27
@justlevine justlevine force-pushed the feat/pricing-fields branch from 52bb138 to 36d5a0b Compare August 29, 2022 12:16
@justlevine justlevine marked this pull request as ready for review August 30, 2022 08:54
@justlevine justlevine merged commit fe69948 into AxeWP:develop Aug 30, 2022
@justlevine justlevine deleted the feat/pricing-fields branch August 30, 2022 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test ✔ This PR can be run using repository secrets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant