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

User Onboarding - wiring up setup & signup correctly #5339

Closed
ErisDS opened this issue May 25, 2015 · 2 comments
Closed

User Onboarding - wiring up setup & signup correctly #5339

ErisDS opened this issue May 25, 2015 · 2 comments
Labels
affects:admin Anything relating to Ghost Admin affects:api Affects the Ghost API
Milestone

Comments

@ErisDS
Copy link
Member

ErisDS commented May 25, 2015

This issue relates to the 'Wiring & navigation behaviour' part of the User Onboarding epic #5315


In #5316, I've wired up the 3 setup screens to be accessible via the top navigation items:

However, at the moment if you submit the form on the 2nd screen, you get setup and signed in, and skip the 3rd screen.

What is supposed to happen, is that submitting the form will cause the setup step to happen, but you won't be signed in (or at least you won't be redirected to the admin screen) until you complete or skip step 3.

As mentioned in the Epic there are a couple of different ways to accomplish this functionality. Ideally the setup step is complete after submitting the form on step 2, but it should also be possible to go back and update your details if you want to. This latter part is quite fiddly as it effects both the blog settings and user (rather than a single model). At the moment this is handled via a special API endpoint which does both - this API endpoint could be updated to have an 'edit' version - this API change is detailed in issue #5342.

If you exit the flow after completing step 2 and come back later, or refresh, you should be taken to the admin screen and skip step 3. The invite user screen is a convenience part of the flow, but is not required so it doesn't make sense to force users back to this screen.

@ErisDS ErisDS added affects:admin Anything relating to Ghost Admin affects:api Affects the Ghost API labels May 25, 2015
@ErisDS ErisDS added this to the Current Backlog milestone May 26, 2015
@ErisDS
Copy link
Member Author

ErisDS commented May 26, 2015

I've updated this issue to link to a secondary issue for the API change, so that the work can be done separately if desired, alternatively one PR can close both issues.

@ErisDS
Copy link
Member Author

ErisDS commented Jul 7, 2015

Part of this work has been done, and this issue has become unclear. Therefore closing it in favour of #5526

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects:admin Anything relating to Ghost Admin affects:api Affects the Ghost API
Projects
None yet
Development

No branches or pull requests

1 participant