-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat(premium): offline premium upsell #490
Conversation
PocketKit: Coverage: 82.48
Generated by 🚫 Danger Swift against a1f5c3f |
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.
Hey @timc-mozilla , as shared on Slack, I am still seeing the regular premium upgrade view after going offline, is this expected?
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.
Awesome @timc-mozilla !! Approving this, we just need to follow up on updating the screen when connection status changes while still viewing the screen, but we can totally follow up with another PR!
ZStack(alignment: .topTrailing) { | ||
if viewModel.annualName.isEmpty { | ||
PremiumUpgradeButton(isYearly: true) | ||
if !viewModel.isOffline { |
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.
The online view could potentially be extracted into its own view for more readability, but it's nothing blocking and we can address it in a separate PR
Summary
References
Implementation Details
Test Steps
PR Checklist:
Screenshots