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

Show upgrade button on the /plans/<site> for Starter and Pro plans #64481

Conversation

claudiucelfilip
Copy link
Contributor

@claudiucelfilip claudiucelfilip commented Jun 9, 2022

Proposed Changes

This shows the Upgrade button next to the Business and eCommerce plan if the current plan is Starter or Pro.

Addresses 807-gh-Automattic/martech

Testing Instructions

  • Go to /plans/[site] on a Starter Plan site
  • The Business and Ecommerce plans should show the Upgrade button

Screenshot 2022-06-09 at 12 41 57

  • Go to /plans/[site] on a Pro Plan site
  • The Business and Ecommerce plans should show the Upgrade button

Screenshot 2022-06-09 at 12 41 35

@claudiucelfilip claudiucelfilip self-assigned this Jun 9, 2022
@github-actions
Copy link

github-actions bot commented Jun 9, 2022

@matticbot
Copy link
Contributor

This PR modifies the release build for editing-toolkit

To test your changes on WordPress.com, run install-plugin.sh editing-toolkit update/show-starter-pro-upgrade-buttons on your sandbox.

To deploy your changes after merging, see the documentation: PCYsg-mMA-p2

@matticbot
Copy link
Contributor

Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:

App Entrypoints (~3 bytes added 📈 [gzipped])

name           parsed_size           gzip_size
entry-stepper        +40 B  (+0.0%)       +3 B  (+0.0%)

Common code that is always downloaded and parsed every time the app is loaded, no matter which route is used.

Sections (~5 bytes added 📈 [gzipped])

name                            parsed_size           gzip_size
themes                                +40 B  (+0.0%)       +5 B  (+0.0%)
theme                                 +40 B  (+0.0%)       +5 B  (+0.0%)
stats                                 +40 B  (+0.0%)       +5 B  (+0.0%)
site-purchases                        +40 B  (+0.0%)       +5 B  (+0.0%)
signup                                +40 B  (+0.0%)       +5 B  (+0.0%)
settings-writing                      +40 B  (+0.0%)       +5 B  (+0.0%)
settings-security                     +40 B  (+0.0%)       +5 B  (+0.0%)
settings-performance                  +40 B  (+0.0%)       +5 B  (+0.0%)
settings                              +40 B  (+0.0%)       +5 B  (+0.0%)
scan                                  +40 B  (+0.0%)       +5 B  (+0.0%)
purchases                             +40 B  (+0.0%)       +5 B  (+0.0%)
purchase-product                      +40 B  (+0.0%)       +5 B  (+0.0%)
posts-custom                          +40 B  (+0.0%)       +5 B  (+0.0%)
posts                                 +40 B  (+0.0%)       +5 B  (+0.0%)
plugins                               +40 B  (+0.0%)       +5 B  (+0.0%)
plans                                 +40 B  (+0.0%)       +5 B  (+0.0%)
people                                +40 B  (+0.0%)       +5 B  (+0.0%)
migrate                               +40 B  (+0.0%)       +5 B  (+0.0%)
media                                 +40 B  (+0.0%)       +5 B  (+0.0%)
marketplace                           +40 B  (+0.0%)       +5 B  (+0.0%)
marketing                             +40 B  (+0.0%)       +5 B  (+0.0%)
jetpack-search                        +40 B  (+0.0%)       +5 B  (+0.0%)
jetpack-connect                       +40 B  (+0.0%)       +5 B  (+0.0%)
jetpack-cloud-settings                +40 B  (+0.0%)       +5 B  (+0.0%)
jetpack-cloud-pricing                 +40 B  (+0.0%)       +5 B  (+0.0%)
jetpack-cloud-partner-portal          +40 B  (+0.0%)       +5 B  (+0.0%)
jetpack-cloud-agency-dashboard        +40 B  (+0.0%)       +5 B  (+0.0%)
hosting                               +40 B  (+0.0%)       +5 B  (+0.0%)
home                                  +40 B  (+0.0%)       +5 B  (+0.0%)
help                                  +40 B  (+0.0%)       +5 B  (+0.0%)
email                                 +40 B  (+0.0%)       +5 B  (+0.0%)
earn                                  +40 B  (+0.0%)       +5 B  (+0.0%)
domains                               +40 B  (+0.0%)       +5 B  (+0.0%)
checkout                              +40 B  (+0.0%)       +5 B  (+0.0%)
backup                                +40 B  (+0.0%)       +5 B  (+0.0%)
activity                              +40 B  (+0.0%)       +5 B  (+0.0%)
accept-invite                         +40 B  (+0.0%)       +5 B  (+0.0%)

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 (~5 bytes added 📈 [gzipped])

name                                                              parsed_size           gzip_size
async-load-masterbar-cart-masterbar-cart-wrapper                        +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-design-playground                                            +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-design-blocks                                                +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-design                                                       +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-reader-sidebar                                       +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-post-editor-editor-media-modal                       +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-my-sites-current-site-stale-cart-items-notice        +40 B  (+0.1%)       +5 B  (+0.0%)
async-load-calypso-my-sites-current-site-notice                         +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-layout-masterbar-checkout-tsx                        +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-layout-masterbar-checkout                            +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-components-web-preview-component                     +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-blocks-jitm-templates-sidebar-banner                 +40 B  (+0.1%)       +5 B  (+0.0%)
async-load-calypso-blocks-jitm-templates-notice                         +40 B  (+0.1%)       +5 B  (+0.0%)
async-load-calypso-blocks-jitm-templates-default                        +40 B  (+0.1%)       +5 B  (+0.0%)
async-load-calypso-blocks-inline-help-popover                           +40 B  (+0.0%)       +5 B  (+0.0%)
async-load-calypso-blocks-editor-checkout-modal                         +40 B  (+0.0%)       +5 B  (+0.0%)

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.
Gzip Size: Compressed size of the JS and CSS files. This much data needs to be downloaded over network.

Generated by performance advisor bot at iscalypsofastyet.com.

@claudiucelfilip claudiucelfilip added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jun 9, 2022
Copy link
Contributor

@gmovr gmovr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works just as expected! LGTM 🚀

@claudiucelfilip claudiucelfilip merged commit f2a170a into update/plan-overhaul-revert Jun 9, 2022
@claudiucelfilip claudiucelfilip deleted the update/show-starter-pro-upgrade-buttons branch June 9, 2022 12:07
@github-actions github-actions bot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jun 9, 2022
claudiucelfilip added a commit that referenced this pull request Jun 15, 2022
…64481)

* Add Starter and Pro plan Business and Ecommerce upgrade whitelists

* Add Pro plan to Business upgrade whitelist
claudiucelfilip added a commit that referenced this pull request Jun 16, 2022
…64481)

* Add Starter and Pro plan Business and Ecommerce upgrade whitelists

* Add Pro plan to Business upgrade whitelist
claudiucelfilip added a commit that referenced this pull request Jul 18, 2022
…64481)

* Add Starter and Pro plan Business and Ecommerce upgrade whitelists

* Add Pro plan to Business upgrade whitelist
claudiucelfilip added a commit that referenced this pull request Jul 19, 2022
…64481)

* Add Starter and Pro plan Business and Ecommerce upgrade whitelists

* Add Pro plan to Business upgrade whitelist
claudiucelfilip added a commit that referenced this pull request Jul 20, 2022
* Disable Pro and Starter config flags

* Removing this check for now in the renew spec-- as the legacy plan notice will be removed (#64400)

* Fix/pro plan reference in etk (#64476)

* Disable Pro and Starter config flags

* Removing this check for now in the renew spec-- as the legacy plan notice will be removed (#64400)

* Swapping out the copy for pro plan in editing toolkit

Co-authored-by: Claudiu Filip <claudiucelfilip@gmail.com>
Co-authored-by: Even Tobiesen <52675688+gmovr@users.noreply.github.com>

* Show upgrade button on the /plans/<site> for Starter and Pro plans (#64481)

* Add Starter and Pro plan Business and Ecommerce upgrade whitelists

* Add Pro plan to Business upgrade whitelist

* Swapping out some text in the FAQ section that mentions the Pro plan (#64475)

* Update isEligibleForProPlan selector to check if siteId is Pro as well (#64457)

* Add plan check to the Pro plan eligibility checker

* Add server-friendly plan check

* Remove unnecessary and faulty current plan fetch

* Plans: show notice for sites on unsupported plans (#64478)

* Update plan references in Marketing (#64470)

* Updated pro reference for Cloudflare UpsellNudge

* Updated the UpsellNudge title with Premium plan

* Updated reference from Pro in the SEO upsell. Note that the findFirstSimilarPlanKey() gives the wrong URL -- which is the same for these other upsells.

* Updated reference from Pro plan in Facebook block in Marketing>Tools

* Remove an extra space in copy

* Updated test for the seo-settings form

* Update link in UpsellNudge (#64496)

* Updating the CTA urls to go to the plans page, as they used to do

* Updating the specific UpsellNudge in the Settings > General form

* Updating a test assertion for form-general.jsx to expect Business

* Removing the check for the eligibleForProPlan here, as this will return false and we won't enter the conditional. (#64511)

* Update/plan overhaul revert seo upsell preview (#64502)

* Updating this nudge to Business Plan

* Updating test for SEO preview upgrade nudge

* Update plans references in upgrade nudges and CTAs (#64484)

* Revert copy and plan in domain-to-plan-nudge to legacy (Personal)

* Update copy in admin-sections in inline-help

* Update contextual help texts from Pro to legacy

* Changing the advertising-removed to reflect legacy plans (Business CTA)

* Update reference to Personal plan required in DomainProductPrice, renderFreeWithPlanText

* Update reference from Pro to Personal on domain search results domain mapping

* Update copy in transfer domain step to legacy

* Update theme upsell popover heading from Pro to legacy

* Update backup upsell card from Pro to legacy

* Update PurchaseDetail on thank you view from Pro to legacy/Business

* Change Business plan upsell nudge (checkout) back to Business from Pro

* Change upsell CTA in AdsWrapper to Business

* Updated payment blocks/membership nudge from Pro to Personal

* Updated CTA/nudge for hosting features from Pro to Business

* Updated upgrade prompt on plugin install page to refer to Business plan

* Fixed upgrade prompts/nudges in media-library from Pro to legacy

* Updated footer card for migration confirmation step from Pro to Business

* Updated reference to Business from Pro for mention on import or migrate step

* Updated reference on migration upsell from Pro to Business

* Updated reference to Business from Pro plan in scan/wpcom-upsell.tsx

* Update upsell reference to Business instead of Pro

* Updated reference from Pro to Personal in site-settings/podcasting-details

* Changed text in UpsellNudge in stats-module under my-sites

* Updated plans references in UpsellNudge(s) under themes

* Updated title in upsellBanner for Jetpack

* Updated reference in footer for blocks>import>ready

* Updated plan reference in CTA for store intent

* Updated strings in store features

* Updated CTAs in focused launch

* Reverting to existing strings for i18n

* Added namespaced classname to button for lint

* swapping out text for pro plan references in Settings --> Performance (#64564)

* Change Pro plan mention on domain search (#64615)

* Use starter plan copy overrides in all sitations

* Mention paid annual plans instead of just paid

* Fix: Upgrade now (to plans page) sometimes takes the user back to my-plan (#64606)

* Remove unnecessary redirect and Plans navigations tab hide

* Remove overhauled plans grid from the Plans page

* Roll back domain search copy before Pro plan overhaul (#64618)

* Restore old copy versions

* Ensure transfer price is not shown for new annual plans copy

* Remove Pro plan from top storage plan list (#64657)

* Plans overhaul: revert Pro upsell in posts list (#64659)

* Remove tos-related files

* Remove unused variable

* Small lint fix

* Add previous admin section copy changes to new file

* Skip Free to Pro test suite

* User better method to disable Free to Pro upgrade test

* Updated the tooltip/description for live chat support to not set a specific day/time range for more flexibility (#65721)

* Starter can upgrade to Premium, and Pro Monthly can upgrade to Business or Ecommerce Monthly (#65724)

* Starter can upgrade to Premium. Pro monthly can upgrade to Business or eCommerce monthly

* Monthly Pro plan can upgrade to higher annual and bi-annual plans

* Wrap legacy notice text in translation check

* Plan Overhaul revert: Do not offer Starter plan to install premium plugins (#65772)

* Use the Goals URL when declining Business Upsell (#65773)

Co-authored-by: Even Tobiesen <52675688+gmovr@users.noreply.github.com>
Co-authored-by: Jessie Harris <jessie.harris@automattic.com>
Co-authored-by: Taegon Kim <gonom9@gmail.com>
Co-authored-by: Christos <chriskmnds@gmail.com>
Co-authored-by: Miguel Torres <miguel.torres@automattic.com>
claudiucelfilip added a commit that referenced this pull request Jul 20, 2022
…64481)

* Add Starter and Pro plan Business and Ecommerce upgrade whitelists

* Add Pro plan to Business upgrade whitelist
claudiucelfilip added a commit that referenced this pull request Jul 21, 2022
* Disable Pro and Starter config flags

* Removing this check for now in the renew spec-- as the legacy plan notice will be removed (#64400)

* Fix/pro plan reference in etk (#64476)

* Disable Pro and Starter config flags

* Removing this check for now in the renew spec-- as the legacy plan notice will be removed (#64400)

* Swapping out the copy for pro plan in editing toolkit

Co-authored-by: Claudiu Filip <claudiucelfilip@gmail.com>
Co-authored-by: Even Tobiesen <52675688+gmovr@users.noreply.github.com>

* Show upgrade button on the /plans/<site> for Starter and Pro plans (#64481)

* Add Starter and Pro plan Business and Ecommerce upgrade whitelists

* Add Pro plan to Business upgrade whitelist

* Swapping out some text in the FAQ section that mentions the Pro plan (#64475)

* Update isEligibleForProPlan selector to check if siteId is Pro as well (#64457)

* Add plan check to the Pro plan eligibility checker

* Add server-friendly plan check

* Remove unnecessary and faulty current plan fetch

* Plans: show notice for sites on unsupported plans (#64478)

* Update plan references in Marketing (#64470)

* Updated pro reference for Cloudflare UpsellNudge

* Updated the UpsellNudge title with Premium plan

* Updated reference from Pro in the SEO upsell. Note that the findFirstSimilarPlanKey() gives the wrong URL -- which is the same for these other upsells.

* Updated reference from Pro plan in Facebook block in Marketing>Tools

* Remove an extra space in copy

* Updated test for the seo-settings form

* Update link in UpsellNudge (#64496)

* Updating the CTA urls to go to the plans page, as they used to do

* Updating the specific UpsellNudge in the Settings > General form

* Updating a test assertion for form-general.jsx to expect Business

* Removing the check for the eligibleForProPlan here, as this will return false and we won't enter the conditional. (#64511)

* Update/plan overhaul revert seo upsell preview (#64502)

* Updating this nudge to Business Plan

* Updating test for SEO preview upgrade nudge

* Update plans references in upgrade nudges and CTAs (#64484)

* Revert copy and plan in domain-to-plan-nudge to legacy (Personal)

* Update copy in admin-sections in inline-help

* Update contextual help texts from Pro to legacy

* Changing the advertising-removed to reflect legacy plans (Business CTA)

* Update reference to Personal plan required in DomainProductPrice, renderFreeWithPlanText

* Update reference from Pro to Personal on domain search results domain mapping

* Update copy in transfer domain step to legacy

* Update theme upsell popover heading from Pro to legacy

* Update backup upsell card from Pro to legacy

* Update PurchaseDetail on thank you view from Pro to legacy/Business

* Change Business plan upsell nudge (checkout) back to Business from Pro

* Change upsell CTA in AdsWrapper to Business

* Updated payment blocks/membership nudge from Pro to Personal

* Updated CTA/nudge for hosting features from Pro to Business

* Updated upgrade prompt on plugin install page to refer to Business plan

* Fixed upgrade prompts/nudges in media-library from Pro to legacy

* Updated footer card for migration confirmation step from Pro to Business

* Updated reference to Business from Pro for mention on import or migrate step

* Updated reference on migration upsell from Pro to Business

* Updated reference to Business from Pro plan in scan/wpcom-upsell.tsx

* Update upsell reference to Business instead of Pro

* Updated reference from Pro to Personal in site-settings/podcasting-details

* Changed text in UpsellNudge in stats-module under my-sites

* Updated plans references in UpsellNudge(s) under themes

* Updated title in upsellBanner for Jetpack

* Updated reference in footer for blocks>import>ready

* Updated plan reference in CTA for store intent

* Updated strings in store features

* Updated CTAs in focused launch

* Reverting to existing strings for i18n

* Added namespaced classname to button for lint

* swapping out text for pro plan references in Settings --> Performance (#64564)

* Change Pro plan mention on domain search (#64615)

* Use starter plan copy overrides in all sitations

* Mention paid annual plans instead of just paid

* Fix: Upgrade now (to plans page) sometimes takes the user back to my-plan (#64606)

* Remove unnecessary redirect and Plans navigations tab hide

* Remove overhauled plans grid from the Plans page

* Roll back domain search copy before Pro plan overhaul (#64618)

* Restore old copy versions

* Ensure transfer price is not shown for new annual plans copy

* Remove Pro plan from top storage plan list (#64657)

* Plans overhaul: revert Pro upsell in posts list (#64659)

* Remove tos-related files

* Remove unused variable

* Small lint fix

* Add previous admin section copy changes to new file

* Skip Free to Pro test suite

* User better method to disable Free to Pro upgrade test

* Updated the tooltip/description for live chat support to not set a specific day/time range for more flexibility (#65721)

* Starter can upgrade to Premium, and Pro Monthly can upgrade to Business or Ecommerce Monthly (#65724)

* Starter can upgrade to Premium. Pro monthly can upgrade to Business or eCommerce monthly

* Monthly Pro plan can upgrade to higher annual and bi-annual plans

* Wrap legacy notice text in translation check

* Plan Overhaul revert: Do not offer Starter plan to install premium plugins (#65772)

* Use the Goals URL when declining Business Upsell (#65773)

* Disable Starter plan testsuite

Co-authored-by: Even Tobiesen <52675688+gmovr@users.noreply.github.com>
Co-authored-by: Jessie Harris <jessie.harris@automattic.com>
Co-authored-by: Taegon Kim <gonom9@gmail.com>
Co-authored-by: Christos <chriskmnds@gmail.com>
Co-authored-by: Miguel Torres <miguel.torres@automattic.com>
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.

3 participants