-
-
Notifications
You must be signed in to change notification settings - Fork 629
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
improve receipt types #1461
improve receipt types #1461
Conversation
andresesfm
commented
Aug 10, 2021
- Added isCanceled to amazon properties
- fix lint
- cleaned up types and refactored common code across platforms
- refactor receipt conversion for Amazon
- verbiage
@@ -13,7 +13,7 @@ | |||
"apsl-react-native-button": "3.1.1", | |||
"react": "17.0.1", | |||
"react-native": "0.64.2", | |||
"react-native-iap": "file:..", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any reason why reverting this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I've been testing and it turns out that Metro doesn't pick up on this notation. Reference here: facebook/metro#68 I really wanted it to work, I ended up using wml: https://github.com/wix/wml#readme which is not great but it gets the job done. I really hope they enable linking on Metro at some point. In the meantime, there's no point in pointing to a file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for spotting the issue. Have you also tried this?
Anyway, let's merge this PR
and we may of this afterward.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
4de8c44
to
a5b315b
Compare