-
Notifications
You must be signed in to change notification settings - Fork 335
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
Add Internal support for draft paywall previews #4761
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to me!
) | ||
} | ||
|
||
// swiftlint:disable:next missing_docs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Idk if we should be documenting these public-but-not-public APIs 🤔 WDYT @RevenueCat/coresdk?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm IMO, I think it's fine to not document them. It's not meant for public usage yeah.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM!
Checklist
Motivation
We need a way to show paywalls in the draft state for the Paywall Previews functionality in the RC app
Description
Adds some new
@_spi
-protected public APIs regarding draft paywalls:*
draftPaywallComponents
property inOffering
, to know if there's a draft paywall available.* A new
init
inPaywallView
that adds theuseDraftPaywall
parameter to show the draft paywall instead of the published one.