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

유저정보 입력UI #42

Merged
merged 15 commits into from
Sep 5, 2024
Merged

유저정보 입력UI #42

merged 15 commits into from
Sep 5, 2024

Conversation

jun0811
Copy link
Contributor

@jun0811 jun0811 commented Sep 3, 2024

#29
회원가입 정보 받는 폼입니다
FormItem 컴포넌트화 일단 간단하게 만들어뒀습니다

추후에 주소 검색 기능 붙일 예정입니다

추후 추가했을때 문제가 있을 수 있음
@jun0811 jun0811 changed the title Feature/#29 user info 유저정보 입력UI Sep 3, 2024
Copy link
Contributor

@kichul7493 kichul7493 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

코멘트 확인해주세요~

expect(screen.getByText('상세 주소')).toBeInTheDocument()
})

it('validates nickname field', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

정상 입력의 경우도 테스트하면 좋지 않을까 생각됩니다.

position?: 'static' | 'fixed' | 'absolute'
}

const Header = ({ left, center, right, position = 'static', ...props }: HeaderProps) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

공용으로 사용할 수 있도록 만들어주셨군요👏👏👏
고생하셨습니다

export type CreateMemberFormDataType = z.infer<typeof createMemberFormSchema>

// useSignupForm 훅 정의
export const useSignUpForm = () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

특별한 이유가 없다면 파일 이름과 Hook 이름이 통일되면 좋지 않을까 싶습니다.😊

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵넵 수정해두겠습니다

@jun0811 jun0811 force-pushed the feature/#29-user-info branch from bb4b580 to 61a0eaa Compare September 4, 2024 03:00
@jun0811 jun0811 force-pushed the feature/#29-user-info branch from 043105a to 854c614 Compare September 4, 2024 08:18
@jun0811 jun0811 merged commit eae9342 into dev Sep 5, 2024
1 check passed
@jun0811 jun0811 deleted the feature/#29-user-info branch September 5, 2024 08:39
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.

2 participants