-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
PM-16517: Create personal use plan for additional storage #5205
base: main
Are you sure you want to change the base?
PM-16517: Create personal use plan for additional storage #5205
Conversation
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.
Looks Good
@@ -38,7 +38,7 @@ public Families2019PasswordManagerFeatures() | |||
HasPremiumAccessOption = true; | |||
|
|||
StripePlanId = "personal-org-annually"; | |||
StripeStoragePlanId = "storage-gb-annually"; |
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.
Are you changing the existing storage name , what of the users that has subscribed to it. I am not sure you are able to change a product name when it has subscribers in stripe
New Issues
Fixed Issues
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5205 +/- ##
=======================================
Coverage 43.70% 43.70%
=======================================
Files 1472 1472
Lines 67963 67963
Branches 6161 6161
=======================================
Hits 29703 29703
Misses 36963 36963
Partials 1297 1297 ☔ View full report in Codecov by Sentry. |
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.
Looks good! We might want to also consider bringing back the "business use" checkbox when creating a new org in another PR. That checkbox should still be there, I think, but just hidden on the client side.
…rate-product-personal-use' into PM-16517-Additional-storage-separate-product-personal-use
🎟️ Tracking
https://bitwarden.atlassian.net/browse/PM-16517
📔 Objective
Separate price & product for personal use Password Manager plans when additional storage is purchased.
To validate still internally whether we update existing subscriptions when listening to webhooks or we update them all at once in bulk.
📸 Screenshots
⏰ Reminders before review
🦮 Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or ℹ️ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or 💭 (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or:warning:
) for more significant problems or concerns needing attention:seedling:
) or ♻️ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes