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

V3 api #3

Merged
merged 131 commits into from
Aug 18, 2020
Merged

V3 api #3

merged 131 commits into from
Aug 18, 2020

Conversation

thecodecafe
Copy link
Collaborator

This branch's main purpose was to implement the v3 API, although several other things followed, at the core, it was for implementing the v3 API.

Here are the things you will find in this branch.

  1. FlutterwaveButton has been renamed to PayWithFlutterwave.
  2. PayWithFlutterwave uses the v3 AP by default so I in order so to use the v2 API variant you will need to import PayWithFlutterwaveV2.
  3. FlutterwaveInit uses the v3 API by default so in order to use the v2 API variant you will need to import FlutterwaveInitV2.
  4. DefaultButton has been changed to FlutterwaveButton.
  5. The contribution guide has been rewritten to something that is simple and that works.
  6. The default documentation is for v3 API.
  7. The ReadMe makes reference to the v2 API ReadMe, there is a v2 API variant of every doc page
  8. Tests have been rewritten for the V2 and V3 components and function.

We initially planned to support only v3 API going forward but then decided to structure the library in such a way we can still render support for those still on the v2 API just in case we have bugs and fixes that needed to be made in the future.

This merge includes the local testing feature for contributors.
This component carries most of the similar code both pay with flutterwave v2 and v3 components
share.
@corneliusyaovi corneliusyaovi merged commit 5d816c7 into master Aug 18, 2020
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.

None yet

2 participants