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 #1558

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Conversation

NachoSoto
Copy link
Contributor

Equivalent to RevenueCat/purchases-ios#3577

I've also added test coverage for errors in PaywallViewModel.

Before

Before

After

After

Copy link

codecov bot commented Jan 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (abbbfd4) 84.17% compared to head (19acd7a) 84.17%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1558   +/-   ##
=======================================
  Coverage   84.17%   84.17%           
=======================================
  Files         218      218           
  Lines        7199     7199           
  Branches     1007     1007           
=======================================
  Hits         6060     6060           
  Misses        748      748           
  Partials      391      391           

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

Copy link
Contributor

@tonidero tonidero left a comment

Choose a reason for hiding this comment

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

Looks great!

return
}

assertThat(state.errorMessage).isEqualTo("Error 10: Error performing request.")
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

@NachoSoto NachoSoto merged commit aeb374a into main Jan 17, 2024
9 checks passed
@NachoSoto NachoSoto deleted the paywalls-improve-error-messages branch January 17, 2024 14:50
NachoSoto pushed a commit that referenced this pull request Jan 17, 2024
**This is an automatic release.**

### RevenueCatUI
* `Paywalls`: improve error display (#1558) via NachoSoto (@NachoSoto)
* Fix paywall dialog composable sizing issues (#1556) via Toni Rico
(@tonidero)
* `Paywalls`: improve landscape layout on all templates (#1551) via
NachoSoto (@NachoSoto)
### Bugfixes
* Fix AmazonConfiguration for Java projects (#1554) via Cesar de la Vega
(@vegaro)
### Dependency Updates
* Bump fastlane-plugin-revenuecat_internal from `0ddee10` to `e6ba247`
(#1557) via dependabot[bot] (@dependabot[bot])

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
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