Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add a registration/set-up page for new admins #3225

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.

Sign up for GitHub

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

Jump to bottom
Closed
3 tasks
presleyp opened this issue Jul 26, 2022 · 5 comments · Fixed by #3476
Closed
3 tasks

Add a registration/set-up page for new admins #3225

presleyp opened this issue Jul 26, 2022 · 5 comments · Fixed by #3476
Assignees
@BrunoQuaresma
Labels
site Area: frontend dashboard

Comments

@presleyp
Copy link
Contributor

presleyp commented Jul 26, 2022 •
edited
Loading

Problem

Admins of new deployments can only set up their first user through the CLI

Solution

In a deployment with no users, the login page redirects to a setup page that has a UI for the setup process

Definition of Done

  • In a deployment with no users, the login page redirects to a setup page with a form
  • The form takes a username, email, and password
  • Submitting the form creates a user and logs them in

Notes

Here's what the signup flow looks like from the CLI: https://asciinema.org/a/cy8QVTPwazDAV6cxcV6DF1Qnj
Here's where initial user creation is implemented in the CLI: https://github.com/coder/coder/blob/v0.8.1/cli/login.go#L81-L196
@presleyp presleyp mentioned this issue Jul 26, 2022
Closed
@presleyp presleyp added site Area: frontend dashboard feature labels Jul 26, 2022
@kylecarbs kylecarbs mentioned this issue Jul 26, 2022
Closed
2 tasks
@tjcran tjcran mentioned this issue Jul 29, 2022
Closed
25 tasks
@BrunoQuaresma
Copy link
Collaborator

I think this issue is maybe not blocked by #3223 but related to it. Is that right?

@BrunoQuaresma BrunoQuaresma self-assigned this Aug 10, 2022
@BrunoQuaresma
Copy link
Collaborator

I also think this is a 5, because of testing. Is it ok if I change it?

@presleyp
Copy link
Contributor Author

@BrunoQuaresma fine to change the estimate! I was thinking it was blocked because the other ticket handles detection of whether the first user has been created, which is needed for this.

@BrunoQuaresma
Copy link
Collaborator

Ahh I see, the detection is done in the API already so no need to have a blocker.

@presleyp
Copy link
Contributor Author

The API call exists, but it isn't called in the frontend yet, right? If you want to just do them both together, you can skip the part about writing about the CLI. The idea was just that it's urgent to make it more clear, and #3223 was enough to do that. It's less urgent to get the full fix in (this ticket).

@BrunoQuaresma BrunoQuaresma mentioned this issue Aug 11, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BrunoQuaresma BrunoQuaresma

Labels
site Area: frontend dashboard
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Add setup page coder/coder
2 participants
@presleyp @BrunoQuaresma