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.

Sign up for GitHub

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

Jump to bottom

[FE][Feat] #59 : AuthModal 컴포넌트 #177

Merged
merged 11 commits into from
Nov 17, 2024
Merged

[FE][Feat] #59 : AuthModal 컴포넌트 #177

merged 11 commits into from
Nov 17, 2024

Conversation

juwon5272
Copy link
Collaborator

📝 PR 개요

  • 모달 열기 닫기 hook 제작
  • 컴포넌트의 common Modal 컴포넌트 구현
    • Modal과 같이 다양한 형태가 나올 수 있는 컴포넌트는 합성 방식의 컴포넌트를 사용하였습니다
    • Input과 같이 비교적 단순한 형태의 컴포넌트에는 구성 방식의 컴포넌트를 사용하였습니다
  • common Modal 컴포넌트를 이용하여 AuthModal 컴포넌트 구현
    • type에 따라 login 또는 register 모달창이 되는 컴포넌트입니다.
  • 각각의 컴포넌트에 대해 storybook을 작성하였습니다

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#59 로그인 모달 창은 ID와 PW를 받을 수 있다.

📷 스크린샷 및 동영상

2024-11-17.12.44.52.mov

📚 참고 자료 (선택 사항)

[Storybook으로 React 컴포넌트 문서화하기]

(https://velog.io/@jackson5272/Storybook%EC%9C%BC%EB%A1%9C-%EC%BB%B4%ED%8F%AC%EB%84%8C%ED%8A%B8-%EB%AC%B8%EC%84%9C%ED%99%94%ED%95%98%EA%B8%B0)

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 16, 2024
leedongyull
leedongyull approved these changes Nov 17, 2024
View reviewed changes
Copy link
Collaborator

@leedongyull leedongyull left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

effozen
effozen approved these changes Nov 17, 2024
View reviewed changes
Copy link
Collaborator

@effozen effozen left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@juwon5272 juwon5272 added 작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 17, 2024
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 17, 2024
@juwon5272 juwon5272 merged commit 557873c into frontend Nov 17, 2024
8 of 15 checks passed
@juwon5272 juwon5272 deleted the feature/fe/#59-modal-login branch November 17, 2024 14:58
@juwon5272 juwon5272 removed the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 17, 2024
happyhyep
happyhyep reviewed Nov 17, 2024
View reviewed changes
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

헙 고생하셨습니다!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@happyhyep happyhyep happyhyep left review comments

@leedongyull leedongyull leedongyull approved these changes

@effozen effozen Awaiting requested review from effozen

Assignees

@juwon5272 juwon5272

Labels
작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@juwon5272 @leedongyull @happyhyep @effozen