-
Notifications
You must be signed in to change notification settings - Fork 82
Expose getStorefront APIs in CustomEntitlementComputation flavor #2579
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
Conversation
a7a9da9
to
142265f
Compare
method @Deprecated public static void getSubscriptionSkusWith(com.revenuecat.purchases.Purchases, java.util.List<java.lang.String> skus, optional kotlin.jvm.functions.Function1<? super com.revenuecat.purchases.PurchasesError,kotlin.Unit> onError, kotlin.jvm.functions.Function1<? super java.util.List<? extends com.revenuecat.purchases.models.StoreProduct>,kotlin.Unit> onReceiveSkus); | ||
method public static void getVirtualCurrenciesWith(com.revenuecat.purchases.Purchases, optional kotlin.jvm.functions.Function1<? super com.revenuecat.purchases.PurchasesError,kotlin.Unit> onError, kotlin.jvm.functions.Function1<? super com.revenuecat.purchases.virtualcurrencies.VirtualCurrencies,kotlin.Unit> onSuccess); |
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.
I'm confused why these are not updated nor found in the checks...
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.
Yea it seems like we cannot rely on the check at all. I think we might need to go the git diff route?
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.
Yup indeed
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2579 +/- ##
=======================================
Coverage 78.64% 78.64%
=======================================
Files 296 296
Lines 10888 10888
Branches 1520 1520
=======================================
Hits 8563 8563
Misses 1663 1663
Partials 662 662 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Looks good!
method @Deprecated public static void getSubscriptionSkusWith(com.revenuecat.purchases.Purchases, java.util.List<java.lang.String> skus, optional kotlin.jvm.functions.Function1<? super com.revenuecat.purchases.PurchasesError,kotlin.Unit> onError, kotlin.jvm.functions.Function1<? super java.util.List<? extends com.revenuecat.purchases.models.StoreProduct>,kotlin.Unit> onReceiveSkus); | ||
method public static void getVirtualCurrenciesWith(com.revenuecat.purchases.Purchases, optional kotlin.jvm.functions.Function1<? super com.revenuecat.purchases.PurchasesError,kotlin.Unit> onError, kotlin.jvm.functions.Function1<? super com.revenuecat.purchases.virtualcurrencies.VirtualCurrencies,kotlin.Unit> onSuccess); |
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.
Yea it seems like we cannot rely on the check at all. I think we might need to go the git diff route?
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! Thank you!
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.
I'm curious: what are these txt files for?
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.
They have the public API for our libraries, both in the defaults flavor and the cec flavor. We're using metalava to generate them. Unfortunately, the check task itself doesn't seem to be working, so we might need to fallback to always dump these api files and compare using git.
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.
These are "dumps" of our public API surface. We have them in other repos too, like purchases-kmp. The idea being that any PR would check if the committed text file is still valid. If not, the PR author should regenerate the file, which would add the diff to the PR, allowing us to review it.
In purchases-android we use Metalava for this (a Google tool), but the "check" task doesn't seem to work consistently. It overlooks changes to the public API, which becomes evident once we regenerate the files and see unrelated changes. (Virtual Currency stuff in this PR for instance.)
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.
(Ha, I should have refreshed before hitting send 😅)
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.
I understand now. Thank you both so much! 🫶
(two explanations is better than one 😄)
While CEC already had the `storefrontCountryCode`, this also adds the APIs `getStorefrontCountryCode` and `awaitStorefrontCountryCode` to the CEC flavor.
**This is an automatic release.** ## RevenueCat SDK ### 🐞 Bugfixes * Add lifetime badge to customer center (#2565) via Cesar de la Vega (@vegaro) ### Customer Center #### ✨ New Features * Introduce custom actions for customer center (#2572) via Facundo Menzella (@facumenzella) ## RevenueCatUI SDK ### ✨ New Features * Add support for az-AZ locale (#2575) via Franco Correa (@francocorreasosa) ### Customer Center #### 🐞 Bugfixes * Display latest expired subscription if no active subscriptions (#2564) via Cesar de la Vega (@vegaro) ### 🔄 Other Changes * Expose getStorefront APIs in CustomEntitlementComputation flavor (#2579) via Toni Rico (@tonidero) * [AUTOMATIC][Paywalls V2] Updates paywall-preview-resources submodule (#2578) via RevenueCat Git Bot (@RCGitBot) * Add missing dokka-hide-internal tests to CI and fix reported number of tests (#2569) via Cesar de la Vega (@vegaro) * [AUTOMATIC][Paywalls V2] Updates paywall-preview-resources submodule (#2574) via RevenueCat Git Bot (@RCGitBot) * Convert WebBilling products into TestStoreProducts (#2573) via Toni Rico (@tonidero) * Add WebBillingGetProducts endpoint (#2571) via Toni Rico (@tonidero) * Remove TestStoreProduct purchase check (#2570) via Toni Rico (@tonidero) * Improve 9.0.0 changelog (#2568) via Toni Rico (@tonidero) * [AUTOMATIC][Paywalls V2] Updates paywall-preview-resources submodule (#2567) via RevenueCat Git Bot (@RCGitBot) * Parallel test jobs (#2549) via Cesar de la Vega (@vegaro) Co-authored-by: revenuecat-ops <ops@revenuecat.com>
While CEC already had the `storefrontCountryCode`, this also adds the APIs `getStorefrontCountryCode` and `awaitStorefrontCountryCode` to the CEC flavor.
Description
While CEC already had the
storefrontCountryCode
, this also adds the APIsgetStorefrontCountryCode
andawaitStorefrontCountryCode
to the CEC flavor.