feat: Add setup page #3476

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

Merged

BrunoQuaresma merged 19 commits into main from bq/3225

Aug 11, 2022

Collaborator

BrunoQuaresma commented Aug 11, 2022

Closes #3225

Screen.Recording.2022-08-11.at.11.47.07.mov

BrunoQuaresma and others added 6 commits

August 10, 2022 13:28


          Check if has first user

2eccbf1


          Add missing handler

a3fabe3


          Add setup

5d3701b


          Make user login after creation

87b55cc


          Authenticate user when setup is done

d6fe749


          Fix setup flow

f81942d

BrunoQuaresma self-assigned this

BrunoQuaresma requested a review from a team as a code owner

August 11, 2022 14:51

Kira-Pilot reviewed

View reviewed changes

scripts/develop.sh Outdated Show resolved Hide resolved

Kira-Pilot reviewed

View reviewed changes

site/src/api/api.ts Show resolved Hide resolved

Kira-Pilot reviewed

View reviewed changes

site/src/components/SignInLayout/SignInLayout.tsx Outdated Show resolved Hide resolved

Kira-Pilot reviewed

View reviewed changes

site/src/components/SignInLayout/SignInLayout.tsx Outdated Show resolved Hide resolved

Kira-Pilot reviewed

View reviewed changes

site/src/components/Welcome/Welcome.tsx Outdated Show resolved Hide resolved

Kira-Pilot reviewed

View reviewed changes

site/src/components/Welcome/Welcome.tsx Outdated Show resolved Hide resolved

Kira-Pilot reviewed

View reviewed changes

site/src/pages/LoginPage/LoginPage.test.tsx Outdated Show resolved Hide resolved


          Apply suggestions from code review

7b37e0e

Co-authored-by: Kira Pilot <kira@coder.com>

Kira-Pilot reviewed

View reviewed changes

site/src/pages/SetupPage/SetupPage.test.tsx Show resolved Hide resolved

Bruno Quaresma added 3 commits

August 11, 2022 15:40


          Add comment into hasFirtUser

4e4008f


          Move to language object

a0ef036


          Refactor tests to not use spy

186a37c

Kira-Pilot reviewed

View reviewed changes

site/src/xServices/auth/authXService.ts Show resolved Hide resolved

Bruno Quaresma added 2 commits

August 11, 2022 15:53


          Merge

e50648f


          Merge branch 'bq/3225' of github.com:coder/coder into bq/3225

2cdebbd

BrunoQuaresma requested a review from ammario as a code owner

August 11, 2022 15:53

Bruno Quaresma added 2 commits

August 11, 2022 15:54


          Merge branch 'main' of github.com:coder/coder into bq/3225

d88d470


          Add back first user on dev script

42fd362

Kira-Pilot approved these changes

View reviewed changes

Member

Kira-Pilot left a comment

I think this looks fantastic; thanks for answering all my questions!

bpmct reviewed

View reviewed changes

site/src/pages/SetupPage/SetupPage.tsx Outdated Show resolved Hide resolved


          Update site/src/pages/SetupPage/SetupPage.tsx

5d988cd

Co-authored-by: Ben Potter <ben@coder.com>

ammario requested changes

View reviewed changes

Member

ammario left a comment

I reviewed from a copy perspective.

site/src/api/api.ts Outdated Show resolved Hide resolved

site/src/pages/SetupPage/SetupPage.tsx Outdated Show resolved Hide resolved

site/src/pages/SetupPage/SetupPageView.tsx Outdated Show resolved Hide resolved

site/src/xServices/setup/setupXService.ts Outdated Show resolved Hide resolved

BrunoQuaresma and others added 2 commits

August 11, 2022 13:22


          Apply suggestions from code review

7959e44

Co-authored-by: Ammar Bandukwala <ammar@ammar.io>


          Better handle hasFirstUser error

3895a0b

BrunoQuaresma requested a review from ammario

August 11, 2022 16:26


          Fix formatting

b5e0a63

ammario approved these changes

View reviewed changes


          Fix login machine

4d13c28

BrunoQuaresma enabled auto-merge (squash)

August 11, 2022 17:21

BrunoQuaresma merged commit 4a17e0d into main

BrunoQuaresma deleted the bq/3225 branch

August 11, 2022 17:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet