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

ensure checkEligibility doesn't refresh the receipt #1265

Merged
merged 1 commit into from Feb 7, 2022

Conversation

aboedo
Copy link
Member

@aboedo aboedo commented Feb 7, 2022

checkTrialOrIntroDiscountEligibility is meant to be called programmatically, and should therefore never trigger a refresh receipt request, since that could make App Store prompt for log in credentials.
This PR ensures that we never refresh the receipt for that method.

This is the v4 equivalent of #1264

@aboedo aboedo requested a review from a team February 7, 2022 18:03
@aboedo aboedo self-assigned this Feb 7, 2022
Copy link
Contributor

@NachoSoto NachoSoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏🏻

@aboedo aboedo merged commit f7e9265 into main Feb 7, 2022
@aboedo aboedo deleted the fix/check_intro_eligibility_causes_login_prompt branch February 7, 2022 18:41
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