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

On-/offboarding copy updates #39055

Merged
merged 20 commits into from Jul 10, 2023
Merged

On-/offboarding copy updates #39055

merged 20 commits into from Jul 10, 2023

Conversation

mdperez86
Copy link
Contributor

@mdperez86 mdperez86 commented Jun 30, 2023

Submission Review Guidelines:

Changes proposed in this Pull Request:

Closes #38960

How to test the changes in this Pull Request:

Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:

New users flow. Note that for this flow the store should not has any published product.

  1. Make sure the New product editor feature is active under /wp-admin/admin.php?page=wc-settings&tab=advanced&section=features
  2. Make sure woocommerce_block_product_tour_shown is set to no and woocommerce_product_editor_show_feedback_bar is set to yes under /wp-admin/tools.php?page=woocommerce-admin-test-helper so WCA Test Helper should be installed
  3. Visit /wp-admin/admin.php?page=wc-admin&path=/add-product you should see the welcome modal with title Meet the product editing form and description Discover the product form's unique features with a quick overview of what's included.
  4. After clicking View highlights button you should see a tour modal with title Fresh and modern interface, description Everything you need to create and sell your products, all in one place. From photos and descriptions to pricing and inventory, all of your product settings can be found here.

 and the illustration shared in second AC from [Enhancement] On-/offboarding copy updates #38960
  5. Clicking the Next button a second modal should be shown with title Content-rich product descriptions and description Show off what's great about your products and engage your customers with content-rich product descriptions. Add images, videos, and any other content they might need to make a purchase.
  6. Clicking the Next button a third modal should be shown with title Lightning fast performance and description Get your products listed and available for purchase in no time! Our modern technology ensures a reliable and streamlined experience.
  7. Clicking the Next button a four modal should be shown with title More features are on the way and description We're actively working on adding more features to the product form, including the ability to add digital products, variations, and more. Watch this space!
  8. After closing the last modal a bottom bar should slides out from the bottom of the screen and with some background blur. Check the design from [Enhancement] On-/offboarding copy updates #38960
  9. Clicking Share feedback link from the bottom bar should show a feedback modal with tile What do you think of the new product form?
  10. When clicking turn it off link from the bottom bar should show a new modal with the following checkboxes
    image and the primary button should be Send

Old users flow. Note that for this flow the store should has at least one published product.

  1. Make sure the New product editor feature is active under /wp-admin/admin.php?page=wc-settings&tab=advanced&section=features
  2. Make sure woocommerce_block_product_tour_shown is set to no and woocommerce_product_editor_show_feedback_bar is set to yes under /wp-admin/tools.php?page=woocommerce-admin-test-helper so WCA Test Helper should be installed
  3. Visit /wp-admin/admin.php?page=wc-admin&path=/add-product you should see the welcome modal with title Welcome to the new product form! and description Discover its new features and improvements with a quick overview of what's included.

Changelog entry

  • Automatically create a changelog entry from the details below.

Significance

  • Patch
  • Minor
  • Major

Type

  • Fix - Fixes an existing bug
  • Add - Adds functionality
  • Update - Update existing functionality
  • Dev - Development related task
  • Tweak - A minor adjustment to the codebase
  • Performance - Address performance issues
  • Enhancement

Message

Comment

@mdperez86 mdperez86 requested a review from a team June 30, 2023 19:40
@mdperez86 mdperez86 self-assigned this Jun 30, 2023
@github-actions github-actions bot added focus: react admin [team:Ghidorah] package: @woocommerce/customer-effort-score issues related to @woocommerce/customer-effort-score plugin: woocommerce Issues related to the WooCommerce Core plugin. labels Jun 30, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 30, 2023

Hi @louwie17,

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:
https://github.com/woocommerce/woocommerce/wiki/Writing-high-quality-testing-instructions

@github-actions
Copy link
Contributor

github-actions bot commented Jun 30, 2023

Test Results Summary

Commit SHA: dca1433

Test 🧪Passed ✅Failed 🚨Broken 🚧Skipped ⏭️Unknown ❔Total 📊Duration ⏱️
API Tests25900202610m 54s
E2E Tests1900018020814m 28s

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.

Copy link
Contributor

@louwie17 louwie17 left a comment

Choose a reason for hiding this comment

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

Nice work on this @mdperez86, this tested well.
I left one comment in relation to the changelog for the CES package, if you could update that.
Also it looks like the Guide in the design uses a primary variant for the next button, I know this wasn't explicitly mentioned in the current issue.
So I can create a separate issue for it, but it should be a relatively simple fix if you could add it.
It just be matching the original Guide component in our copied version for this line: https://github.com/WordPress/gutenberg/blob/trunk/packages/components/src/guide/index.tsx#L154
Here is our version:

Significance: minor
Type: enhancement

New product block editor modal text enhancements
Copy link
Contributor

Choose a reason for hiding this comment

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

We should change this changelog to be CES specific.
In this case we are fixing the onsubmit label.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done -> 743725a

@mdperez86 mdperez86 requested a review from louwie17 July 10, 2023 15:07
Copy link
Contributor

@louwie17 louwie17 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 Thanks for the updates, this still tested well, thanks :)

@mdperez86 mdperez86 merged commit 96c0c5b into trunk Jul 10, 2023
18 checks passed
@mdperez86 mdperez86 deleted the enhancement/38960 branch July 10, 2023 18:51
@github-actions github-actions bot added this to the 8.0.0 milestone Jul 10, 2023
@mattsherman
Copy link
Contributor

@mdperez86 @louwie17 This PR broke the alignment of the feedback bar content... it used to line up with the form (same left/right margins). It now is offset...

Screenshot 2023-07-13 at 00 06 02

@mattsherman
Copy link
Contributor

It also broke the positioning of the snackbar notices... they should appear above the feedback bar (they did). They no longer do -- they now overlap the feedback bar (they shouldn't).

Screenshot 2023-07-13 at 00 10 33

@mattsherman
Copy link
Contributor

(I can create a new issue for these bugs later -- it's too late in the evening right now... just didn't want to lose track of them)

@louwie17
Copy link
Contributor

(I can create a new issue for these bugs later -- it's too late in the evening right now... just didn't want to lose track of them)

Good catch @mattsherman, I had not noticed those when reviewing. It be nice to get those in before the code freeze on Monday (if possible), especially the alignment one (cc @mdperez86 )

@mattsherman
Copy link
Contributor

@louwie17 @mdperez86 Here is the issue for tracking the regressions...

#39228

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: @woocommerce/customer-effort-score issues related to @woocommerce/customer-effort-score plugin: woocommerce Issues related to the WooCommerce Core plugin.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement] On-/offboarding copy updates
3 participants