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

[FEAT] 로그인 모달 마크업 (#18) #23

Merged
merged 7 commits into from
Jan 19, 2025

Conversation

thay123028
Copy link

✨ PR 세부 내용

  • 마크업과 디자인은 전부 끝냈습니다.
  • 아이디 찾기, 비밀번호 찾기, 간편 로그인, 회원가입 기능은 이후에 개발하겠습니다.
  • 로그인 버튼 클릭 시 아이디와 비밀번호 필드가 비었는지 확인하고 서버로 요청을 보냅니다.
  • 요청 시 이슈 19번에서 생성한 request 사용하였습니다.

📝 관련 이슈

⚠️주의 사항

💬 리뷰 요청 사항

⏳ 기한

@thay123028 thay123028 changed the title [FEAT] 로그인 모달 마크업 [FEAT] 로그인 모달 마크업 (#18) Jan 2, 2025
@thay123028 thay123028 changed the base branch from main to develop January 2, 2025 17:47
Copy link
Contributor

@aong13 aong13 left a comment

Choose a reason for hiding this comment

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

확인했습니다! LGTM

마이페이지 관련된 기능도 함께 올라온 것 같아서, pr과 관련된 파일만 push되도록 주의해주세요! pr이 밀려있어서 그런 것 같기도 하네요

@@ -58,11 +68,12 @@ function Header() {
</ProfileSection>
) : (
<ProfileSection>
<ProfileNavItem to="/login">로그인</ProfileNavItem>
<ProfileNavItem onClick={handleOpenLoginModal}>로그인</ProfileNavItem>
Copy link
Contributor

Choose a reason for hiding this comment

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

모달 여는 동작 확인했습니다

@aha-rin
Copy link

aha-rin commented Jan 11, 2025

확인했습니다

Copy link

@Rlriiii Rlriiii left a comment

Choose a reason for hiding this comment

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

확인했습니다!!

@aong13 aong13 merged commit 077bfe1 into develop Jan 19, 2025
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.

4 participants