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

Fixed bug with rejection on ios #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

the-wunmi
Copy link

@the-wunmi the-wunmi commented Jan 24, 2020

Hello @tolu360 , good job with this project!

There is a bug around cardParamsAreValid block when a charge is attempted with invalid card details, and expected error is thrown, subsequent trials always return Another request is still being processed, please wait. It is caused because requestIsCompleted was not updated when the initial error was thrown; this PR fixes that.

This may in part fix #38 (comment)

Cheers! :)

Copy link

@sokoya sokoya left a comment

Choose a reason for hiding this comment

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

@the-wunmi This fix it. Thank you!

@secosan
Copy link

secosan commented Aug 13, 2020

Hi all!

Really appreciate your work on this.

Any idea when the PR will be approved by second proxy, merged and released? We are waiting on this same issue on a project.

Thank you

@IyiolaOwabumowa
Copy link

Paystack how does this take eternity, some of us have clients!

This is so unfair

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.

react-native-paystack for RN 0.60* and below
4 participants