Skip to content

Performance Improvement | Documentation Update #54

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

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

jeremiahseun
Copy link

I made few changes that makes development relatively easy. The following:

  • Change Transaction Charge Type value (From String to enum)
  • Change the Payment Option to a List (We won't have to guess what type of payment option are available. Now, it is a list that you add whatever option you want to add.
  • Upgrade the Example Compile SDK Version (From 32 to 33, so as the example app will run fine in Android SDK 33).
  • Few changes and correction - Optimization purposes.

- I changed transactionChargeType as a String to an enum. It will be easier for someone who is not familiar with the type of transaction charge type, to be able to use it without confusion.
- I also noticed that the previous Contribution guidelines link is preventing everyone from accessing it. So I changed it to the Contribution guidelines available.
- I changed the way we input the payment option. As it is inputted into a list, it converts to a String as before. It is just easier for devs.
- I added assertions to cross-check that both paymentOptions and paymentOptionsList are not non-null.
Upgrade the Example Compile SDK Version to 33 and changed the GradleException to FileNotFoundException as it is not used again.
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.

1 participant