woocommerce / woocommerce-admin

(Deprecated) This plugin has been merged to woocommerce/woocommerce
https://woocommerce.github.io/woocommerce-admin/#/
Other
361 stars 146 forks source link

Home Screen: Enable store setup task list if user skips the wizard #4989

Closed pmcpinto closed 3 years ago

pmcpinto commented 4 years ago

Currently, the store setup task list is only enabled if the user doesn't skip the wizard. However, it can be valuable to automatically enable the task list when WC is activated:

joshuatf commented 4 years ago

Estimate: 1

Though there could be some follow-up issues if there is any task list logic dependent on profiler (though at a glance it doesn't appear there will be any issues here).

samueljseay commented 3 years ago

This one is actually a bug, because most of the time the task list does display.

I'll paraphrase these findings reported by @becdetat (thank you!)

The only place the store setup task list does not show is if the onboarding wizard is cancelled via "use the new setup experience" step which is not shown if you're using the woocommerce-admin plugin.

If you skip the onboarding wizard from the store details step it still shows the task list. verified that woocommerce_task_list_complete and woocommerce_task_list_hidden are both left unset after skipping the onboarding wizard, which should have triggered the task list being visible, but somehow the task list wasn’t visible and the quicklinks are visible. The code for that is here.

joshuatf commented 3 years ago

Seems like the enable_onboarding which acted as a param to modify the feature flag is no longer necessary and the root of this issue. On the WCA side, we just need to clean up references to this.

A follow-up is needed in core to make sure the "Not right now" action is actually able to skip the profiler.

joshuatf commented 3 years ago

On the WC core side, I think this issue will be resolved by bumping the WCA version number, though it does look like some clean-up may be needed around setup wizard files no longer used there.