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 and improve support for PMPro (Paid Memberships Pro) #954

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

bernhardkaindl
Copy link

While Tutor supports PMPro as a monetization engine, the current implementation of get_orders_by_user_id() returns an empty orders array.

Fix this function to return orders captured in PMPro, implemented and tested by Joshua Davis (@a11smiles)

  • Add condition for PMPro and WooCommerce (both return shop_order for the post type).
  • Add support for querying PMPro subscriptions
  • Make the second inner join optional (for PMPro orders, the meta key _is_tutor_order_for_course isn't saved).

Was originally opened by Joshua Davis [email protected] @a11smiles as #419,

  • commits and log messages squashed,
  • rebased to current dev
  • and spaces at end of lines fixed by me.

Originally-From: Joshua Davis [email protected]

While Tutor supports PMPro as a monetization engine, the current
implementation of get_orders_by_user_id() returns an empty orders array.

Fix this function to return orders captured in PMPro, implemented
and tested by Joshua Davis (@a11smiles)

- Add condition for PMPro and WooCommerce (both return shop_order
  for the post type).
- Add support for querying PMPro subscriptions
- Make the second inner join optional (for PMPro orders, the meta key
  _is_tutor_order_for_course isn't saved).

Was originally opened by Joshua Davis <[email protected]> @a11smiles as themeum#419,
- commits and log messages squashed,
- rebased to current dev
- and spaces at end of lines
fixed by me.

Originally-From: Joshua Davis <[email protected]>
@bernhardkaindl bernhardkaindl changed the base branch from master to dev August 7, 2024 13:22
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.

2 participants