Skip to content
This repository has been archived by the owner on Mar 30, 2022. It is now read-only.

feature/mantine form with validation for host signup #9

Merged
merged 2 commits into from
Feb 27, 2022

Conversation

remusplesa
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Feb 27, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/host-a-refugee/host-a-refugee/JCU6UzvdjPLgjPbGhWoXAvFVdqk8
✅ Preview: https://host-a-refugee-git-feature-validated-form-host-a-refugee.vercel.app

@Seth-McKilla Seth-McKilla changed the title mantine form with validation for host signup Feat/mantine form with validation for host signup Feb 27, 2022
@Seth-McKilla Seth-McKilla changed the title Feat/mantine form with validation for host signup feature/mantine form with validation for host signup Feb 27, 2022
Copy link
Contributor

@Seth-McKilla Seth-McKilla left a comment

Choose a reason for hiding this comment

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

Since this route is protected and required user to be logged in I think it would be good to just use the user's session info from next-auth for name and email when submitting form instead of having them manually fill this out.

<Title order={3}>Find a host</Title>
<Space h="lg" />

<form onSubmit={form.onSubmit(onSubmitHandler)}>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we just use the session.user.name from NextAuth for this instead of having them fill it out?

label="First Name"
required
/>
<TextInput
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we just use the session.user.name from NextAuth for this instead of having them fill it out?

label="Phone Number"
required
/>
<TextInput
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we just use the session.user.email from NextAuth for this instead of having them fill it out?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants