Added next-auth for login #1
Conversation
assets/draw.png
Outdated
There was a problem hiding this comment.
I suggest using SVGs over PNG format whenever possible. SVGs can be scaled without losing the quality of the image.
Read more about it here: https://cloudinary.com/guides/image-formats/svg-vs-png-4-key-differences-and-how-to-choose#1-image-quality-and-scaling
src/components/signup/signup.tsx
Outdated
There was a problem hiding this comment.
Let's leverage react-hook-form with custom hooks(if needed) to DRY-up things
src/components/input/input.tsx
Outdated
| export default function Input({ | ||
| handleChange, | ||
| value, | ||
| labelText, | ||
| labelFor, | ||
| id, | ||
| name, | ||
| type, | ||
| isRequired = false, | ||
| placeholder, | ||
| customClass, | ||
| }: Partial<InputProps>) { |
There was a problem hiding this comment.
I'd suggest reconsidering the usage and consumption of this component and based on that, can we please come up with a more tightly-coupled and fail-safe component?
There was a problem hiding this comment.
I'd suggest reconsidering the composition of this component. Can we please come up with a more tightly-coupled and fail-safe component?
…ame wrapper and loading page
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…dleware, correct types
…rved keywords in next-auth, hence have to change "user" model name to "userModel"
No description provided.