-
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
Live Preview with Bundle Themes #86274
Comments
wrt to bundled themes: For the For the checkout link - yes, they all need the business plan or higher - "bundled" means "installing this theme will also install some plugins", and plugins are all on the business plan |
For theme tiers, then use of theme tiers will simplify/genericise some of this logic, we can just check "the lowest plan necessary to use a theme of this tier", for example. We're still some weeks away from releasing to customers though and as the live preview is part of the block-editor package it makes it hard to use feature flags inside of the live preview itself. @rcrdortiz has been doing the work of updating the live preview to work with tiers. |
Thank you for providing helpful info, @dsas! @Automattic/lego
In light of this, I have decided to move this issue to the Backlog for the time being. Other resources;
|
Description
Lego shipped Live Preview in Premium and WooCommerce themes, as detailed in #79223. However, it is currently limited to Premium and Woo themes, while it should be expanded to all Bundle themes that are coming (See #85182 (comment), p6rkRX-75y-p2).
Several changes are required at this moment to facilitate this transition to Bundle themes-applicable. Here are some key areas to consider:
wp-calypso/apps/wpcom-block-editor/src/wpcom/features/live-preview/hooks/use-can-preview-but-need-upgrade.ts
Lines 44 to 47 in 7d5dd4f
wp-calypso/apps/wpcom-block-editor/src/wpcom/features/live-preview/hooks/use-can-preview-but-need-upgrade.ts
Lines 222 to 230 in 7d5dd4f
wp-calypso/apps/wpcom-block-editor/src/wpcom/features/live-preview/hooks/use-previewing-theme.ts
Lines 76 to 77 in 7d5dd4f
useBundleSettings
?Aligning with the Theme Tiers project paYJgx-4mO-p2 might be necessary, as I saw some code comments. However, I haven't fully explored this aspect yet. Relevant code segments include:
wp-calypso/apps/wpcom-block-editor/src/wpcom/features/live-preview/hooks/use-can-preview-but-need-upgrade.ts
Lines 36 to 39 in 7d5dd4f
wp-calypso/apps/wpcom-block-editor/src/wpcom/features/live-preview/hooks/use-previewing-theme.ts
Lines 25 to 28 in 7d5dd4f
wp-calypso/apps/wpcom-block-editor/src/wpcom/features/live-preview/hooks/use-previewing-theme.ts
Lines 38 to 41 in 7d5dd4f
Acceptance Criteria
Dev Tasks
TBD
Testing
TBD
Related
CC: @Automattic/lego @renatho @Automattic/t-rex
The text was updated successfully, but these errors were encountered: