-
Notifications
You must be signed in to change notification settings - Fork 84
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
Paywall Template 7 (Siamese) not recognized (react-native-purchases-ui 7.28.1) #1044
Comments
👀 We've just linked this issue to our internal tracker and notified the team. Thank you for reporting, we're checking this out! |
Hi @Tom-Standen, |
Hey @mshmoustafa, thanks for the heads up. My mistake for sounding the alarm, I thought it was a bug! Best, |
No worries, you wouldn't have known without asking! |
@geoffcfchen what version of our SDK are you using? You should be on 8.0.1 |
Hi, @mshmoustafa After updating to 8.0.1, ios version does work. However, android does not work. when I use the following code to print out offerings.all
for ios, I got
But for android, I got
You can see obviously "Standard" is missing in android, which I use it for template 7. |
@geoffcfchen I think this issue is separate from paywalls, from what I understand getOfferings doesn't depend on your paywall template. Can you run this curl request with the same app user ID and API key that you use in your code and let me know what the response is?
|
Hi @mshmoustafa Thank you for helping out. here is the output
|
Hi @mshmoustafa I found that Android returned the correct offerings without missing any objects today. Maybe you guys have fixed it? thanks. for ios, I got
For Android, NOW I got
So the first problem was fixed. However, although Android works fine when I use offerings.all["Default"] (template 5), Android shows the following error when I switch to offerings.all["Standard"] (template 7). Just to let you know that both offerings.all["Default"] and offerings.all["Standard"] work fine in ios. ERROR Your app just crashed. See the error below. |
@geoffcfchen we didn't fix anything related to this so I think it was just a broken cache, sandbox glitches, etc. It's common for that to happen in testing. |
@geoffcfchen Hey! The multi-tier paywall isn't available until React Native version |
@joshdholtz, Hey. Thanks for the information. Just to note that I am using "react-native-purchases": "8.0.1", |
Ah, sorry! Coming back from being sick this weekend so not fully in it yet 🫠 Looking harder this time and will come back with something actually useful 😇 |
@geoffcfchen Okay, now after really looking at the crash, it seems like you might not have the correct version of Do you happen to know if you have this dependency pinned at a lower version? This should be found in Jetpack Compose version |
@joshdholtz, we are using Expo Managed Workflow right now (we do not have a plan to eject from Expo Managed Workflow yet), so we're somewhat restricted in how we can use native Android dependencies like androidx directly due to the managed workflow... Follow up: |
Just to report here. After updating to the 8.1.0, the Android version works now. |
Thought I would use this thread instead of creating a new issue as it's related. Great job on Siamese - it's perfect for our use case so thank you! One quick issue though. We seem to be seeing the package id come through on our template instead of the dynamic '{{ sub_period }} Plan' that we were hoping for. I assumed that this was due to TestFlight or dev bugs but we are now in production and seeing this still on our PayWall. In the short term I will recreate all packages and give them ids of exactly how I want them to appear int the app but thought you guys would want to know. Production PayWall RevCat Console 1 RevCat Packages |
@Tom-Standen let me check on that |
@Tom-Standen just letting you know that the engineering team is looking into it! |
@Tom-Standen a fix is out here! RevenueCat/purchases-ios#4273. It is available in v8.2.3 of the React Native SDK: https://github.com/RevenueCat/react-native-purchases/releases/tag/8.2.3 |
Describe the bug
I have moved to using Paywall 7 - Siamese from Paywall 5 - Bengal as it is the perfect use for our application. However, when I run the same code as before but with the new paywall I see an error that makes it look like the sdk isn't prepared for 7 - Siamese yet.
However, I am receiving the following error:
ERROR TEXT:
Template not recognized: 7.
You can fix this by editing the paywall in the RevenueCat dashboard
The displayed paywall contains default configuration
This error will be hidden in production.
Additional context
Here is a screenshot of the error:
Here is the function that is causing the error:
The text was updated successfully, but these errors were encountered: