Skip to content

fix(next): ensures create-first user works#6020

Merged
jmikrut merged 6 commits into
betafrom
fix/create-first-user-pt2
Apr 25, 2024
Merged

fix(next): ensures create-first user works#6020
jmikrut merged 6 commits into
betafrom
fix/create-first-user-pt2

Conversation

@jmikrut
Copy link
Copy Markdown
Member

@jmikrut jmikrut commented Apr 24, 2024

Description

  1. Ensures create-first-user works properly
  2. Ensures email / password / confirm password are not cleared when creating a new user in admin UI
  • I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

) {
if (formState.password) result.password = formState.password
if (formState.email) result.email = formState.email
if (formState.password) result.password = formState.password
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra if here

Comment thread test/auth/e2e.spec.ts
Comment on lines -30 to -44
const createFirstUser = async ({ page, serverURL }: { page: Page; serverURL: string }) => {
await page.goto(serverURL + '/admin/create-first-user')
await page.locator('#field-email').fill(devUser.email)
await page.locator('#field-password').fill(devUser.password)
await page.locator('#field-confirm-password').fill(devUser.password)
await page.locator('#field-custom').fill('Hello, world!')

await wait(500)

await page.locator('.form-submit > button').click()

await expect
.poll(() => page.url(), { timeout: POLL_TOPASS_TIMEOUT })
.not.toContain('create-first-user')
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove this?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i did not do this, someone else must have. picked up on the branch after it was already in progress

@jmikrut jmikrut merged commit 5f7af53 into beta Apr 25, 2024
@jmikrut jmikrut deleted the fix/create-first-user-pt2 branch April 25, 2024 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants