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][Fix] #407, #410, #413 : 버그 수정 #414

Merged
merged 12 commits into from
Dec 4, 2024
Merged

[FE][Fix] #407, #410, #413 : 버그 수정 #414

merged 12 commits into from
Dec 4, 2024

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Dec 4, 2024 •
edited
Loading

📝 PR 개요

버그 수정

🔍 변경 사항

  • Alert 눌렀을 때 로딩 멈추는 문제 해결
  • 로그인 모달에서 회원가입 모달로 넘겼다가, X 버튼 눌러서 모달 닫고 로그인 시 회원가입 모달로 넘가는 문제 수정
  • 로그인 버튼 포커스 수정

✅ 체크리스트 (Checklist)

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

🔄 관련 이슈 (Linked Issues)

#407, #410, #413

📷 스크린샷 및 동영상 (선택 사항)

2024-12-04.1.59.18.mov
2024-12-04.11.38.07.mov
2024-12-04.2.02.20.mov

📚 참고 자료

ProgressBar 버그 수정기
모바일 페이지에서 클릭 후 Hover가 유지되는 경우

happyhyep and others added 10 commits December 3, 2024 22:34
@happyhyep
Merge pull request #385 from boostcampwm-2024/frontend
caa7adb 
6주차 2차 배포
@effozen
[FE][Fix] #376 : 흰색 배경 안나오는 거 수정
fd29f60
@happyhyep
Merge pull request #389 from boostcampwm-2024/frontend
3b40b4f 
6주차 3차 배포
@happyhyep
Merge pull request #391 from boostcampwm-2024/frontend
a5cb3d5 
6주차 3차 배포
@happyhyep
Merge pull request #393 from boostcampwm-2024/frontend
ba7e34d 
[FE][Fix] guest의 위치 초기값 받아오지 못하는 문제 해결
@happyhyep
Merge pull request #402 from boostcampwm-2024/frontend
c6d1e49 
6주차 4차 배포
@happyhyep
Merge pull request #404 from boostcampwm-2024/frontend
55c9375 
[FE][Fix] 불필요한 콘솔, 주석 제거
@effozen
[FE][Fix] #407 : Alert를 길게 눌렀을 때 로딩 멈추는 문제 해결
6ddb80f
@effozen
[FE][Fix] #410 : 로그인 -> 회원가입 후 닫기, 다시 로그인 창 올 시 회원가입 상태가 유지되는 문제 수정
d885f41
@effozen
[FE][Fix] #413 : 로그인 -> 회원가입 후 닫기, 다시 로그인 창 올 시 회원가입 상태가 유지되는 문제 수정
5ce0fdc
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Dec 4, 2024
@happyhyep happyhyep self-requested a review December 4, 2024 05:04
@happyhyep happyhyep self-requested a review December 4, 2024 06:21
happyhyep
happyhyep approved these changes Dec 4, 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.

고생하셨습니다!!

@happyhyep happyhyep merged commit 6c09ecd into frontend Dec 4, 2024
4 checks passed
@effozen effozen deleted the feature/fe/#407-alert-error branch December 5, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@happyhyep happyhyep happyhyep approved these changes

@juwon5272 juwon5272 juwon5272 approved these changes

@leedongyull leedongyull Awaiting requested review from leedongyull

Assignees

@effozen effozen

Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Milestone
No milestone
Development

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