FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Patch 1 by owati · Pull Request #52 · iamraphson/react-paystack · GitHub

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 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 closed this Apr 28, 2022
owati reopened this Apr 28, 2022

iamraphson commented Aug 6, 2022
edited
Loading

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 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 commented Aug 30, 2022

Copy link
Copy Markdown
Author

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL