-
Notifications
You must be signed in to change notification settings - Fork 1
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
Partial points checkout #5
Labels
Comments
Closed
JDGrimes
added a commit
that referenced
this issue
Mar 8, 2017
Otherwise it will result in fatal errors like this: ``` [08-Mar-2017 15:03:35 UTC] PHP Fatal error: Uncaught Error: Call to undefined function wc_get_order() in /srv/git/_wordpoints-modules/woocommerce/src/classes/hook/action/order/delete.php:23 Stack trace: #0 /srv/git/wordpoints/src/classes/hook/router.php(147): WordPoints_WooCommerce_Hook_Action_Order_Delete->should_fire() #1 /srv/git/wordpoints/src/classes/hook/router.php(94): WordPoints_Hook_Router->route_action('delete_post,10', Array) #2 /srv/www/wordpress-develop/src/wp-includes/class-wp-hook.php(298): WordPoints_Hook_Router->__call('delete_post,10', Array) #3 /srv/www/wordpress-develop/src/wp-includes/class-wp-hook.php(323): WP_Hook->apply_filters(NULL, Array) #4 /srv/www/wordpress-develop/src/wp-includes/plugin.php(453): WP_Hook->do_action(Array) #5 /srv/www/wordpress-develop/src/wp-includes/post.php(2508): do_action('delete_post', '143') #6 /srv/www/wordpress-develop/src/wp-includes/post.php(6048): wp_delete_post('143', true) #7 /srv/www/wordpress-develop/src/wp-includes/class-wp-hook.php(298): wp_delete_auto_drafts() #8 /srv/www/wordpress-develo in /srv/git/_wordpoints-modules/woocommerce/src/classes/hook/action/order/delete.php on line 23 ``` If WooCommerce isn't active, our events and actions won't operate properly, and neither will our entities, etc. So there is no since in registering them.
Maybe just coupons would be a better way of doing it. Otherwise, how would we handle refunds, for example? |
I don't at a glance see any plugins that allow multiple different payment options to be used. I think that myCred uses coupons under the hood, which is an interesting option. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It would be nice to have the ability to pay part with points and part with another method. The points would act like a coupon, reducing the price. We could also let the admin put a limit on how much of a purchase may be payed for with points (e.g., 20%).
The text was updated successfully, but these errors were encountered: