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

4.0.0-beta.5 #288

Merged
merged 12 commits into from
Jan 30, 2025
Merged

4.0.0-beta.5 #288

merged 12 commits into from
Jan 30, 2025

Conversation

yusuftor
Copy link
Collaborator

@yusuftor yusuftor commented Jan 30, 2025

Changes in this pull request

Breaking Changes

  • Replaces entitlements.status with subscriptionStatus.
  • entitlementStatusDidChange reverted to subscriptionStatusDidChange.
  • EntitlementStatus removed in favor of SubscriptionStatus.

Enhancements

  • Adds Superwall.shared.subscriptionStatus.isActive as a convenience variable.

Checklist

  • All unit tests pass.
  • All UI tests pass.
  • Demo project builds and runs.
  • I added/updated tests or detailed why my change isn't tested.
  • I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
  • I have run swiftlint in the main directory and fixed any issues.
  • I have updated the SDK documentation as well as the online docs.
  • I have reviewed the contributing guide

yusuftor and others added 12 commits January 15, 2025 16:54
Removed check for whether transaction was purchased within last hour as this seemed to be causing issues in Testflight where the spinner wouldn't disappear. It could be that the transactionDate that's returned for that wasn't playing ball.
- Adds `Superwall.shared.subscriptionStatus.isActive` convenience variable.
- Moves away from EntitlementStatus to SubscriptionStatus.
fixes variant assignment maths and adds unit tests
@yusuftor yusuftor merged commit b0b0e4e into v4 Jan 30, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants