-
Notifications
You must be signed in to change notification settings - Fork 292
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
StoreProduct
: add localized price per period strings
#3546
Conversation
723aa73
to
884b54e
Compare
nvmd, those aren't equivalent since they're not string |
ab57c5d
to
f1533d3
Compare
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.
🚢
Only thing missing is docstrings
@@ -442,6 +442,15 @@ class StoreProductTests: StoreKitConfigTestCase { | |||
self.logger.verifyMessageWasLogged(Strings.storeKit.sk1_no_known_product_type, level: .debug) | |||
} | |||
|
|||
func testPricePerPeriod() async throws { |
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.
<3
StoreProduct
: add localized price per period strings
…ePerYear formatted variants
f1533d3
to
2ccd6f3
Compare
|
@aboedo oh I just noticed you mentioned that! Yeah I removed that code now that it's implemented in |
**This is an automatic release.** ### New Features * `StoreProduct`: add localized price per period strings (#3546) via Andy Boedo (@aboedo) ### RevenueCatUI * `Paywalls`: new `.onPurchaseCancelled` and `paywallViewControllerDidCancelPurchase:` (#3578) via NachoSoto (@NachoSoto) * `Paywalls`: improve error display (#3577) via NachoSoto (@NachoSoto) ### Dependency Updates * Bump fastlane-plugin-revenuecat_internal from `0ddee10` to `e6ba247` (#3575) via dependabot[bot] (@dependabot[bot]) ### Other Changes * `PurchaseTester`: improved `ReceiptInspector` so it accepts receipts with escape sequences (#3554) via Andy Boedo (@aboedo)
Added
localizedPricePerWeek
,localizedPricePerMonth
andlocalizedPricePerYear
.I started on this before noticing that we already have an equivalent at the package level, still need to unify them.