[CYS on core] Update the WooCommerce Extensions Themes page to include references to the CYS #45468
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Test Results SummaryCommit SHA: f884719
To view the full API test report, click here. To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
… 'Design your own' button and their current active theme is not TT4
woocommercebot
requested review from
a team and
imanish003
and removed request for
a team
nefeline
added
focus: customize-your-store
|
Apart from reviewing the code changes, please make sure to review the testing instructions as well. You can follow this guide to find out what good testing instructions should look like: |
nefeline
requested review from
albarin and
gigitux
and removed request for
imanish003
nefeline
changed the title
albarin
approved these changes
Mar 12, 2024
nefeline
deleted the
45418-cys-on-core-update-the-market-place-themes-screen
branch
github-actions
bot
added
the
needs: analysis
alopezari
added
needs: external testing
Konamiman
pushed a commit
that referenced
this pull request
Mar 13, 2024
…e references to the CYS (#45468) * Introduce the new 'Design your own' button in the themes screen. * unify the SCSS margins * Add double quotes for woocommerce-marketplace__sub-header * Add the CYS banner to the marketplace. * Ensure each child in a list should have a unique 'key' prop * Update the NoAIBanner component to direct users to the CYS flow. * Add the customizeStoreDesignUrl const * Ditch navigateOrParent * Add the 'Browse the WordPress.org theme directory to discover more' link. * Ensure the warning Modal is displayed whenever the user clicks on the 'Design your own' button and their current active theme is not TT4 * Remove the unnecessary fragment * Add changefile(s) from automation for the following project(s): woocommerce * Fix lint. --------- Co-authored-by: github-actions <github-actions@github.com>
11 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Submission Review Guidelines:
Changes proposed in this Pull Request:
Here a series of changes are being made to the WooCommerce Extensions Themes page to include references to the new CYS flow, including:
Add the Design your own button. If the current active theme is TT4, it directs users to the assembler, if not, the theme switch warning modal is displayed to confirm the user wants to proceed to the CYS flow and change their current active theme.
Add the Design your own section right after the second row of themes. If the current active theme is TT4, the "Start designing" button directs users to the assembler; if not, the theme switch warning modal is displayed to confirm the user wants to proceed to the CYS flow and change their current active theme. It is also important to note that the position of the banner is dynamically changed on screen resize to ensure it preserves its expected position in the second row of themes independent of the number of columns displayed:
Add the line Didn't find a theme you like? Browse the WordPress.org theme directory to discover more. The link sends the user to Appearance -> Themes -> Add new Theme with e-commerce as a search term.
Closes #45418.
How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
customize-storeis enabledReset Customize Your Storewp-admin/admin.php?page=wc-admin&tab=themes&path=%2FextensionsIt is also important to note that the position of the banner is dynamically changed on screen resize to ensure it preserves its expected position in the second row of themes independent of the number of columns displayed:
Screen.Recording.2024-03-12.at.09.31.54.mov
Changelog entry
Significance
Type
Message
Update the WooCommerce Extensions Theme page to include references to the Customize Your Store flow.
Comment