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

PurchaseTester: ignore errors when restoring purchases #2228

Merged
merged 2 commits into from
Jan 18, 2023

Conversation

NachoSoto
Copy link
Contributor

See #2211.

If restorePurchases fails for any reason, we still want to try to load the local receipt. We're only interested in refreshing the receipt to load it.

See #2211.

If `restorePurchases` fails for any reason, we still want to try to load the local receipt. We're only interested in refreshing the receipt to load it.
@NachoSoto NachoSoto added the test label Jan 18, 2023
@NachoSoto NachoSoto requested a review from a team January 18, 2023 19:30
@NachoSoto NachoSoto enabled auto-merge (squash) January 18, 2023 21:30
@NachoSoto NachoSoto force-pushed the purchase-tester-restore-error branch from bddd6bb to 2158979 Compare January 18, 2023 21:30
@NachoSoto NachoSoto merged commit 64a9f42 into main Jan 18, 2023
@NachoSoto NachoSoto deleted the purchase-tester-restore-error branch January 18, 2023 22:20
NachoSoto pushed a commit that referenced this pull request Jan 19, 2023
**This is an automatic release.**

### Bugfixes
* `Purchases`: avoid potential crash when initializing in the background
(#2231) via NachoSoto (@NachoSoto)
### Other Changes
* `PurchaseTester`: ignore errors when restoring purchases (#2228) via
NachoSoto (@NachoSoto)
* `PurchaseTester`: fixed `isPurchasing` state when purchasing fails
(#2229) via NachoSoto (@NachoSoto)
* `PurchaseTester`: setting `changelog` when submitting to `TestFlight`
(#2232) via NachoSoto (@NachoSoto)
* Revert "`SPM`: added `APPLICATION_EXTENSION_API_ONLY` flag to
`RevenueCat` and `ReceiptParser` (#2217)" (#2225) via NachoSoto
(@NachoSoto)
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.

2 participants