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

Merged repo #1

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Merged repo #1

wants to merge 17 commits into from

Conversation

RocKhalil
Copy link

No description provided.

akshetpandey and others added 17 commits September 27, 2017 12:12
…04#123)

Restricting IAPs doesn't prevent us from fetching product information. A better way to handle this is to check canMakePayments method and show appropriate message on the payment screen.

Update README to include canMakePayments

Also add note about calling canMakePayments before purchaseProduct to show a better error message to users.
Changed the installation steps to use `react-native install` rather than the old `rnpm`
* Improved variable naming in Readme.md

Also added a console.log line just to help people know what should be happening

* Might as well make it const not var
* main queue

* requiresMainQueueSetup: set to NO
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet