Skip to content

Support setting null offering id on PaywallView#2658

Merged
tonidero merged 1 commit into
mainfrom
support-setting-null-offering-id
Sep 4, 2025
Merged

Support setting null offering id on PaywallView#2658
tonidero merged 1 commit into
mainfrom
support-setting-null-offering-id

Conversation

@tonidero

@tonidero tonidero commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Description

Follow-up to #2610. We had a non-null offeringIdentifier parameter, which doesn't allow us to reset the offering selection to none. This would be very uncommon, but we should still support it.

@tonidero tonidero requested review from a team and rickvdl September 4, 2025 08:31
@tonidero tonidero changed the title Support setting null offering id Support setting null offering id on PaywallView Sep 4, 2025
@codecov

codecov Bot commented Sep 4, 2025

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2658   +/-   ##
=======================================
  Coverage   78.09%   78.09%           
=======================================
  Files         306      306           
  Lines       11484    11484           
  Branches     1590     1590           
=======================================
  Hits         8968     8968           
  Misses       1819     1819           
  Partials      697      697           

☔ 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.

@rickvdl rickvdl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@tonidero tonidero added this pull request to the merge queue Sep 4, 2025
Merged via the queue into main with commit 9eb73ce Sep 4, 2025
21 checks passed
@tonidero tonidero deleted the support-setting-null-offering-id branch September 4, 2025 09:37
tonidero added a commit that referenced this pull request Sep 10, 2025
### Description
Follow-up to #2610. We had a non-null offeringIdentifier parameter,
which doesn't allow us to reset the offering selection to none. This
would be very uncommon, but we should still support it.
@tonidero tonidero mentioned this pull request Sep 10, 2025
tonidero pushed a commit that referenced this pull request Sep 10, 2025
**This is an automatic release.**

## RevenueCat SDK
### ✨ New Features
* Add `RevenueCatBackupAgent` (#2625) via Toni Rico (@tonidero)

### 🔄 Other Changes
* Fix CoroutineCreationDuringComposition lint error on AGP 8.13.0
(#2659) via Cesar de la Vega (@vegaro)
* Support setting null offering id on PaywallView (#2658) via Toni Rico
(@tonidero)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants