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

Report restore errors when using PaywallActivityLauncher #1544

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

tonidero
Copy link
Contributor

@tonidero tonidero commented Jan 2, 2024

Description

In the PaywallActivityLauncher result, we were only reporting errors happening during purchasing, but not when an error happened during restores.

This PR fixes that so we also report restore errors as long as that's the last operation that happened before closing the paywall.

@tonidero tonidero marked this pull request as ready for review January 2, 2024 17:23
@tonidero tonidero requested a review from a team January 2, 2024 17:23
Copy link

codecov bot commented Jan 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a0404de) 84.14% compared to head (41156e1) 84.14%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1544   +/-   ##
=======================================
  Coverage   84.14%   84.14%           
=======================================
  Files         218      218           
  Lines        7196     7196           
  Branches     1007     1007           
=======================================
  Hits         6055     6055           
  Misses        750      750           
  Partials      391      391           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tonidero tonidero merged commit 662032b into main Jan 3, 2024
@tonidero tonidero deleted the report-restore-errors-paywall-activity-launcher branch January 3, 2024 11:26
vegaro pushed a commit that referenced this pull request Jan 3, 2024
**This is an automatic release.**

### RevenueCatUI
* PaywallActivityLauncher: Add callback indicating whether the paywall
was displayed when presented conditionally to an entitlement identifier
(#1542) via Toni Rico (@tonidero)
* Report restore errors when using PaywallActivityLauncher (#1544) via
Toni Rico (@tonidero)
* Remove scroll in Template 2 and 5 in Footer mode (#1545) via Cesar de
la Vega (@vegaro)
### Bugfixes
* Opens PaywallFooterView (#1541) via Cesar de la Vega (@vegaro)
### Dependency Updates
* Bump fastlane from 2.217.0 to 2.218.0 (#1543) via dependabot[bot]
(@dependabot[bot])

Co-authored-by: revenuecat-ops <[email protected]>
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.

2 participants