Skip to content
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

Fix iOS crash showing paywall for current offering #129

Merged
merged 5 commits into from
Jul 15, 2024

Conversation

JayShortway
Copy link
Member

As the title says. The commit messages are pretty descriptive. The actual fix is tiny and in the last commit.

Closes #128

@JayShortway JayShortway added the pr:fix A bug fix label Jul 12, 2024
@JayShortway JayShortway requested a review from a team July 12, 2024 13:07
@JayShortway JayShortway self-assigned this Jul 12, 2024
@@ -1,16 +1,16 @@
PODS:
Copy link
Member

Choose a reason for hiding this comment

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

we may not even wanna commit this file, it'll fall out of date super often

Copy link
Member Author

Choose a reason for hiding this comment

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

I see indeed that other hybrids don't have it version controlled either (although PHC does). In general I think it's nice to be a bit more explicit about transitive dependencies, so I'd propose to remove the file once it becomes a problem.

iosApp/Podfile Show resolved Hide resolved
fastlane/Fastfile Outdated Show resolved Hide resolved
@JayShortway JayShortway merged commit 621aad3 into main Jul 15, 2024
5 checks passed
@JayShortway JayShortway deleted the fix-crash-showing-paywall-for-current-offering branch July 15, 2024 11:11
@RCGitBot RCGitBot mentioned this pull request Jul 26, 2024
@RCGitBot RCGitBot mentioned this pull request Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:fix A bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash when displaying paywall without explicitly providing offering
3 participants