[PM-33518] feat: Add isInAppUpgradeAvailableFlow to PremiumStateManager#6795
Conversation
Bitwarden Claude Code ReviewOverall Assessment: APPROVE This PR adds a new No findings. |
|
Great job! No new security vulnerabilities introduced in this pull request |
Combine isInAppBillingSupportedFlow and MobilePremiumUpgrade feature flag into a single StateFlow for determining whether to route premium upgrade CTAs to the in-app PlanScreen or the web vault URL.
eda128d to
581f032
Compare
5d1322c to
65f09c8
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## premium-upgrade/PM-33517-settings-premium_bak #6795 +/- ##
=================================================================================
+ Coverage 85.66% 85.72% +0.06%
=================================================================================
Files 831 828 -3
Lines 59011 58979 -32
Branches 8633 8634 +1
=================================================================================
+ Hits 50553 50562 +9
+ Misses 5468 5427 -41
Partials 2990 2990
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|

🎟️ Tracking
https://bitwarden.atlassian.net/browse/PM-33518
📔 Objective
Add a consolidated
isInAppUpgradeAvailableFlowtoPremiumStateManagerthat combinesisInAppBillingSupportedFlowand theMobilePremiumUpgradefeature flag into a single observable signal.Stacked on PM-33517 PR (settings/premium management)
Changes
isInAppUpgradeAvailableFlow: StateFlow<Boolean>toPremiumStateManagerinterfacePremiumStateManagerImplby combining billing support and feature flag flows