-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Storage Add Ons: Implement Plans 2023 Features Grid Functionality #80992
Conversation
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser: Sections (~5503 bytes added 📈 [gzipped])
Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to. Async-loaded Components (~5309 bytes added 📈 [gzipped])
React components that are loaded lazily, when a certain part of UI is displayed for the first time. Legend What is parsed and gzip size?Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. Generated by performance advisor bot at iscalypsofastyet.com. |
25e04b4
to
7bbedde
Compare
I'll be adding notes to this PR shortly. Hopefully it'll make it easier to code review 🤞 |
Hey @saygunnyc 👋 I have a few design questions. For the storage add-on mobile layout in p9Jlb4-7Ui-p2, there were designs for the dropdown in a plan card Since the dropdown is only shown for business and ecommerce plans, I was wondering if you could elaborate on if we'd expect storage labels for other plan cards ( personal, premium, etc. ) I was also curious if we could figure out how to handle larger cost strings in the storage add on dropdown. My first thought is to round the cost up or down to the nearest dollar, or to hide the cost after selection |
92be47b
to
f2b8bdb
Compare
In order to avoid race conditions in the /start flow, we have to use the flow controller to introduce add-ons to the shopping cart
f3f84dc
to
f2641d6
Compare
f2641d6
to
3079ae0
Compare
3079ae0
to
aa1386c
Compare
Related to #79041 (comment)
Issue https://github.com/Automattic/martech/issues/2025
Tracking https://github.com/Automattic/martech/issues/2023
Proposed Changes
GIFs
Needs to be fixed
Will Follow Up in Future PR
/plans
page. They currently don't add the storage add-on into the checkout cart properly/plans
spotlight cardTesting Instructions
/start?flags=plans%2Fupgradeable-storage
/start/domain?flags=plans%2Fupgradeable-storage
/start/plans-import?flags=plans%2Fupgradeable-storage
/start/onboarding-pm?flags=plans%2Fupgradeable-storage
setup/onboarding-media?flags=plans%2Fupgradeable-storage
/setup/link-in-bio-tld?flags=plans%2Fupgradeable-storage
Pre-merge Checklist