@ancentz Interesting find. We’ll definitely look at correcting that at some point. Thanks for letting us know.
https://github.com/moonstonemedia/WP-Simple-Pay-Lite-for-Stripe/issues/31
Hi Phil,
I have had a look at the Code myself. And i was thinking if it is ok to add the
do_action( 'sc_after_charge', $charge_response );
https://github.com/moonstonemedia/WP-Simple-Pay-Lite-for-Stripe/blob/d957390aba51a2693dcb78607507969988fc3008/classes/class-stripe-checkout-shared-functions.php#L132
-
This reply was modified 9 years, 5 months ago by
ancentz.
@ancentz That’s a good idea, but we already have an action hook sc_after_charge that runs on the payment details page. That’s probably the wrong terminology there, but we have to leave it now.
How about the action hook sc_after_charge_before_redirect in the place you mentioned? Would that work?
Great suggestion, Ill try it out. Thanks for the help Phil appreciate it.
@ancentz Sorry if I wasn’t clear. I meant to ask do you want to try adding the action hook sc_after_charge_before_redirect in the place you indicated?
If you edit the plugin code and that works, then we can add it to be included in the next update.
Alternatively, you can open a pull request in our GitHub repo referencing the issue you opened.