Skip to content

Patch 1#52

Open
owati wants to merge 2 commits into
iamraphson:masterfrom
owati:patch-1
Open

Patch 1#52
owati wants to merge 2 commits into
iamraphson:masterfrom
owati:patch-1

Conversation

@owati

@owati owati commented Apr 15, 2022

Copy link
Copy Markdown

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

iamraphson commented Aug 6, 2022

Copy link
Copy Markdown
Owner

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

@owati

owati commented Aug 8, 2022

Copy link
Copy Markdown
Author

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.

@owati

owati commented Aug 30, 2022

Copy link
Copy Markdown
Author

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

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.

3 participants