Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Fix for Amazon purchase dialog not showing up #552
Fix for Amazon purchase dialog not showing up #552
Changes from 2 commits
3de0b41
a11a295
619967e
4b6faea
0e93706
e8c5a3e
5aa5306
7d9a3fa
51c425b
94d85a6
e1d7863
7792135
bb5cfeb
6bef4de
03cbd27
72659e3
4cde5a3
8a32ed4
dfdc9a4
90f9631
885b343
4fc2c9c
177c964
bafef41
2516c89
9bd8e0f
72672ac
2cbd49c
28ee907
0b147e4
3cf3048
3d03404
3ed4185
22a4aa3
9a17cb2
11dad87
bfed36f
a9f6872
f8bb051
252d825
0872ebb
f641be4
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
So this suggests that we support having multiple purchase requests initiated at the same time, but considering we are using the same PROXY_AMAZON_BILLING_ACTIVITY_REQUEST_CODE, we only really support one (and I think that's fine). So probably we could refactor this to allow only 1 purchase to be initiated at the same time? If we do that, we won't need to be passing a callback and we can call
purchasingServiceProvider.onPurchaseCompleted(activity)
on theonPurchaseResponse
. We would still need to hold a reference to theactivity
here (same as we are doing now, inside the callback), we can make it aWeakReference
... (not a fan of this either...)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.
That's true... We actually error out before getting to this point indicating there's another purchase in progress