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

Paywalls: improve error display #3577

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Conversation

NachoSoto
Copy link
Contributor

@NachoSoto NachoSoto commented Jan 16, 2024

Before:

This prevents "leaking" RevenueCat in the error dialogs:

Simulator Screenshot - iPhone 15 Pro Max - 2024-01-16 at 12 45 13

After:

With this change, users still get some information about the problem, and we get enough context to be able to diagnose it:

Simulator Screenshot - iPhone 15 Pro Max - 2024-01-16 at 12 57 15

This prevents "leaking" _RevenueCat_ in the error dialogs:

With this change, users still get some information about the problem, and we get enough context to be able to diagnose it:
@NachoSoto NachoSoto requested review from a team January 16, 2024 20:58
@NachoSoto NachoSoto enabled auto-merge (squash) January 16, 2024 21:03
NachoSoto added a commit to RevenueCat/purchases-android that referenced this pull request Jan 16, 2024
@NachoSoto NachoSoto merged commit 56741f0 into main Jan 17, 2024
24 checks passed
@NachoSoto NachoSoto deleted the paywalls-improve-error-messages branch January 17, 2024 00:27
NachoSoto pushed a commit that referenced this pull request Jan 17, 2024
**This is an automatic release.**

### New Features
* `StoreProduct`: add localized price per period strings (#3546) via
Andy Boedo (@aboedo)
### RevenueCatUI
* `Paywalls`: new `.onPurchaseCancelled` and
`paywallViewControllerDidCancelPurchase:` (#3578) via NachoSoto
(@NachoSoto)
* `Paywalls`: improve error display (#3577) via NachoSoto (@NachoSoto)
### Dependency Updates
* Bump fastlane-plugin-revenuecat_internal from `0ddee10` to `e6ba247`
(#3575) via dependabot[bot] (@dependabot[bot])
### Other Changes
* `PurchaseTester`: improved `ReceiptInspector` so it accepts receipts
with escape sequences (#3554) via Andy Boedo (@aboedo)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants