Skip to content

Conversation

tonidero
Copy link
Contributor

@tonidero tonidero commented Sep 2, 2025

Checklist

  • If applicable, unit tests
  • If applicable, create follow-up issues for purchases-ios and hybrids

Motivation

Documentation about the Purchases.awaitPurchase coroutine extension was misleading.
Fixes a typo on documentation where the wrong class is mentioned.

Description

Fixed the typo by linking the correct class (`PurchasesError`) embedded in the `PurchasesTransactionException` thrown in case of errors.

Contributed by @matteinn in #2593

…2593)

<!-- Thank you for contributing to Purchases! Before pressing the
"Create Pull Request" button, please provide the following: -->

### Checklist
- [ ] If applicable, unit tests
- [ ] If applicable, create follow-up issues for `purchases-ios` and
hybrids

### Motivation
<!-- Why is this change required? What problem does it solve? -->
<!-- Please link to issues following this format: Resolves #999999 -->
Documentation about the `Purchases.awaitPurchase` coroutine extension
was misleading.
Fixes a typo on documentation where the wrong class is mentioned.

### Description
<!-- Describe your changes in detail -->
<!-- Please describe in detail how you tested your changes -->
Fixed the typo by linking the correct class (`PurchasesError`) embedded
in the `PurchasesTransactionException` thrown in case of errors.
@tonidero tonidero requested a review from a team September 2, 2025 07:23
@tonidero tonidero enabled auto-merge September 2, 2025 07:24
Copy link

codecov bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.53%. Comparing base (dd099d3) to head (4936843).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2651   +/-   ##
=======================================
  Coverage   78.53%   78.53%           
=======================================
  Files         304      304           
  Lines       11362    11362           
  Branches     1577     1577           
=======================================
  Hits         8923     8923           
  Misses       1749     1749           
  Partials      690      690           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tonidero tonidero added this pull request to the merge queue Sep 2, 2025
Merged via the queue into main with commit 3f66c6c Sep 2, 2025
21 checks passed
@tonidero tonidero deleted the external/matteinn/chore/doc-update branch September 2, 2025 10:37
github-merge-queue bot pushed a commit that referenced this pull request Sep 3, 2025
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* Add preferred UI locale override for RevenueCat UI components (#2620)
via Josh Holtz (@joshdholtz)

### 🔄 Other Changes
* Improve thread safety of setting paywalls preferred locale (#2655) via
Josh Holtz (@joshdholtz)
* Remove validation for no packages on paywalls (#2653) via Josh Holtz
(@joshdholtz)
* Video Component Models (dark code) (#2646) via Jacob Rakidzich
(@JZDesign)
* [EXTERNAL] docs: fixed a typo on documentation about
`Purchases.awaitPurchase` by @matteinn in #2593 (#2651) via Toni Rico
(@tonidero)
* Add warning with 9.x issues to all versions since 9.0.0 in CHANGELOG
(#2650) via Toni Rico (@tonidero)
* [AUTOMATIC][Paywalls V2] Updates paywall-preview-resources submodule
(#2647) via RevenueCat Git Bot (@RCGitBot)
* Delete CLAUDE.md (#2648) via Cesar de la Vega (@vegaro)
* MON-1193 flatten Transition JSON structure after chatting more
thoroughly with team (#2641) via Jacob Rakidzich (@JZDesign)

---------

Co-authored-by: revenuecat-ops <ops@revenuecat.com>
Co-authored-by: Josh Holtz <me@joshholtz.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants