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

Expose registration flows #197

Merged
merged 1 commit into from
Jul 29, 2024
Merged

Expose registration flows #197

merged 1 commit into from
Jul 29, 2024

Conversation

dcramer
Copy link
Owner

@dcramer dcramer commented Jul 28, 2024

Add a register and email verification flow, as well as the related supporting components. Refactors a variety of things, including changing the overall design of forms.

Note this also fixes jsx-email, which was broken in dev/prod (passing in tests). There was an issue with react-jsx in typescript which I couldn't figure out, so instead we opted to switch to react for the jsx option. The specific error was React is not defined when executing the code in the server application.

This is missing a reset password flow, which will be a followup.

@dcramer dcramer enabled auto-merge (squash) July 28, 2024 07:53
@dcramer dcramer disabled auto-merge July 28, 2024 08:01
@dcramer dcramer force-pushed the feat/registration branch 12 times, most recently from fb7da9e to 0277c33 Compare July 29, 2024 17:52
Copy link

codecov bot commented Jul 29, 2024

Test Failures Detected: Due to failing tests, we cannot provide coverage reports at this time.

❌ Failed Test Results:

Completed 334 tests with 1 failed, 333 passed and 0 skipped.

View the full list of failed tests

src/worker/jobs/updateCountryStats.test.ts

  • Class name: src/worker/jobs/updateCountryStats.test.ts
    Test name: updates totalBottles

    AssertionError: expected 4 to deeply equal 3
    ❯ .../worker/jobs/updateCountryStats.test.ts:26:36

@dcramer dcramer force-pushed the feat/registration branch 2 times, most recently from 9509443 to fa6e0ae Compare July 29, 2024 18:30
@dcramer dcramer changed the title Add registration and overhaul splash layout Expose registration flows Jul 29, 2024
@dcramer
Copy link
Owner Author

dcramer commented Jul 29, 2024

Sign-in

image
image

Register

image
image

Verification

image
image
image

@dcramer dcramer enabled auto-merge (squash) July 29, 2024 18:50
@dcramer dcramer merged commit 2f92b56 into main Jul 29, 2024
4 checks passed
@dcramer dcramer deleted the feat/registration branch July 29, 2024 18:58
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.

1 participant