-
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
Home Screen: Checklist step asks users to add a menu — when one is created by default already #44240
Comments
This issue is stale because it has been 180 days with no activity. You can keep the issue open by adding a comment. If you do, please provide additional context and explain why you’d like it to remain open. You can also close the issue yourself — if you do, please add a brief explanation and apply one of relevant issue close labels. |
This issue can still be reproduced. |
I believe we should be able to check if a menu is assigned on the user's site and conditionally switch the heading to say "Edit your site menu" |
There are quite a few things going on here simultaneously, and I think the incorrect copy is the least of our problems:
Non-risky actions I see are:
Anything else feels like we'd potentially increase the confusion accidentally. Does that make things better while we wait for FSE sites, which of course will change everything for this task completely? Are there other simple actions we could do to make this much better for soon-to-be-legacy sites? @olaolusoga & @ollierozdarz. cc @autumnfjeld just to follow the convo at this point. |
We just bumped it lower in the list (#49706) It looks like it is trivial to change the copy to "Edit the site menu" in https://github.com/Automattic/wp-calypso/blob/trunk/client/my-sites/customer-home/cards/tasks/site-setup-list/get-task.js#186 |
The title of this issue covers the issue but to reproduce …
Create a new site.
Note the instructions about the checklist in the menu.
Follow instructions and visit the Customizer.
Note that a menu with multiple pages already exists by default.
cc @lcollette
The text was updated successfully, but these errors were encountered: