-
Notifications
You must be signed in to change notification settings - Fork 44
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
[정성혜] week14 #455
The head ref may contain hidden characters: "part2-\uC815\uC131\uD61C-week14"
[정성혜] week14 #455
Conversation
…ithub-actions [Fix] delete merged branch github action
<Controller | ||
name='email' | ||
control={control} | ||
rules={{ | ||
required: '이메일을 입력해주세요.', | ||
pattern: { | ||
value: /\S+@\S+\.\S+/, | ||
message: '올바른 이메일 주소가 아닙니다.', | ||
}, | ||
validate: async (value) => { | ||
const res = await fetch('/api/check-email', { | ||
method: 'POST', | ||
body: JSON.stringify({ email: value }), | ||
}); | ||
const data = await res.json(); | ||
if (!res.ok) { | ||
throw new Error(data.message || '서버 오류입니다.'); | ||
} | ||
if (!data.available) { | ||
throw new Error('이미 사용 중인 이메일입니다.'); | ||
} | ||
}, | ||
}} | ||
render={({ field }) => ( | ||
<Input | ||
{...field} | ||
type='email' | ||
placeholder='이메일을 입력해 주세요.' | ||
error={errors.email?.message} | ||
/> | ||
)} | ||
/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
react-hook-form의 Controller는 register로 제어하기 어려운 컴포넌트를 다룰때 주로 사용되는 래퍼 컴포넌트입니다.
인풋에는 register를 이용해보심이 어떨까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
대신 인풋을 forwardRef를 사용해서 만들어보셔야겠네요!
} | ||
|
||
const Input = ({ | ||
variant, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
현재 variant는 미구현된 기능으로 보여집니다.
이런 경우 주석을 활용해서 추후 개발할 기능이란 의미에서 TODO 주석을 활용해 보심은 어떨까요?
머지를 잘못된 타겟 브랜치로 하셔서 수정해두었습니다. |
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게
이전 주차 미션을 제대로 해결 못해서 원활한 진행이 되지 않았습니다.
아직 next에 대한 이해가 부족하다고 느껴 공부하면서 추후 수정하도록 하겠습니다. 🥲