Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.43k stars 1.99k forks source link

Fixed the upsells on premium themes #96617

Closed rcrdortiz closed 3 days ago

rcrdortiz commented 3 days ago

Related to:

Proposed Changes

Why are these changes being made?

Testing Instructions

https://github.com/user-attachments/assets/f7fcb5fe-65b1-430c-9839-8a0fd1da74a7

Pre-merge Checklist

github-actions[bot] commented 3 days ago
Calypso Live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125273
Jetpack Cloud live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125273&env=jetpack
Automattic for Agencies live (direct link)
https://calypso.live?image=registry.a8c.com/calypso/app:build-125273&env=a8c-for-agencies
matticbot commented 3 days ago

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

rcrdortiz commented 3 days ago

Solves the bug, thank you!

An improvement would be to use different wording about gs because they no longer need separate unlocking (like premium plan themes do)

Personal Premium Screenshot 2024-11-21 at 11 45 28 Screenshot 2024-11-21 at 11 47 31 Could be done as a follow-up though.

I was planning on re-visiting the upgrade pills in this issue: https://github.com/Automattic/dotcom-forge/issues/9676 Before our changes there already were some issues on the DP. I have it marked as a Good first issue so that Antonio can work on it and get familiar with Calypso.

matticbot commented 3 days ago

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

To test WordPress.com changes, run install-plugin.sh $pluginSlug fix/premium-themes-upsell on your sandbox.