-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add a step to ask if the user has an existing store #48
base: master
Are you sure you want to change the base?
Conversation
…completed” action when selected
…e WC already installed (this is a more reliable indicator)
… it should still do something when clicked)
Let's also ping @rickybanister for some opinions from a designer point of view 🎨 |
Could I get a quick walk through of this to better understand how it will flow? It seems fine, but the relationship between the dropdown with the 'yes' option and the 'nope' being a button has me a little confused. |
The dropdown opens to show the options when clicked, following this which @jessefriedman mocked up. It moves to the next step as soon as an option is clicked. Here's a screen recording of part of the flow, from setting up WooCommerce (I skipped the steps only to get to the end of this wizard faster), to the final review screen (the end of JPO). |
Perhaps we could relabel 'nope' to something more like 'I don't need a store'. My issue was the order of reading—I'd forgotten the original question as I evaluated the dropdown and 'next' button so 'nope' felt non-sequitur by then. |
You shouldn't see this unless you installed WooCommerce in the previous step. We could be repetitive and relabel Nope to "No I didn't have a store" or something? |
You're right, but I'm still highly confused. Can I ask, what is the point of this step? Why did they just create a store if they previously had one? If our goal is to do some kind of migration or to just gather data, why wouldn't we put this question before they went off to install WooCommerce? |
Now that I've gone through that step, it's something that bugs me as well. Is the purpose of this step just to collect some data from the users? I also find it confusing from user's perspective to have this step after we've (probably) installed WooCommerce. Other than that, it works as expected, nice work @ryelle. A quick suggestion: I think it would it be useful if we have a text field displayed when the user selects "Other" - just so they can clarify which platform they're using. |
Yeah, that's all we're doing here. I'm not sure if we're looking for the details if someone selects "other", but maybe @jessefriedman can say for sure. I have an issue about the order of steps, #49, if you want to drop any feedback there too. |
Adds a new step after returning from setting up WooCommerce. The positioning of this step is a little awkward - you'll see it if you "Install WooCommerce" then opt out of the WC wizard, or if we detect you already have WC installed.
To test:
@jessefriedman @johnHackworth @roccotripaldi @tyxla