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

Patch 1 #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Patch 1 #52

wants to merge 2 commits into from

Conversation

owati
Copy link

@owati owati commented Apr 15, 2022

Added a pre-payment functionality to the button

owati added 2 commits April 15, 2022 08:20
I was using the paystack button but I wanted the validate the form details before processing payment. In the end I had to make a custom button that made use of the usePaystackPayment hook.
I was using the paystack button but I wanted the validate the form details before processing payment. In the end I had to make a custom button that made use of the usePaystackPayment hook.
@owati owati closed this Apr 28, 2022
@owati owati reopened this Apr 28, 2022
@iamraphson
Copy link
Owner

iamraphson commented Aug 6, 2022

Hi, I don't get the point of this PR. You don't want to be notified about the callback?

@owati
Copy link
Author

owati commented Aug 8, 2022

It is for the button to be able to run a function before calling the payment handler. Like using the button to validate a form then calling the payment function after.

Copy link

@Mellobrainbox Mellobrainbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So how do you want me to merge this deployment of yours

@owati
Copy link
Author

owati commented Aug 30, 2022

I think it is only people with write access that can merge PRs

@Mellobrainbox
Copy link

Yes sir

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.

4 participants