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

[FE][Feature] #113, #114 : 손녀 기준 화면 우측 하단에 가이드 추가 및 드랍다운 버튼 상태 반영 #279

Closed
wants to merge 5 commits into from

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 26, 2024

📝 PR 개요

손녀 기준 화면 우측 하단에 가이드 추가 및 드랍다운 버튼 상태 반영

🔍 변경 사항

  • 드랍다운 버튼 수정 : path 지정 가능하게 함. (상태로 보여줄 경우 수정할 수 있도록 구현)
  • 우측하단 가이드 추가 및 서버에서 받아온 정보로 처리할 수 있도록 구현
  • 페이지별 라우팅 구현

✅ 체크리스트 (Checklist)

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

🔄 관련 이슈 (Linked Issues)

#113, #114

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

2024-11-26.5.32.29.mov

@effozen effozen self-assigned this Nov 26, 2024
@effozen effozen requested review from happyhyep and juwon5272 and removed request for happyhyep November 26, 2024 10:02
@effozen effozen added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 26, 2024
@effozen effozen requested a review from happyhyep November 26, 2024 10:02
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.

저희 사용자 눌렀을 때, 라우팅 변하는 거 아니고 그냥 상태 변화로 하기로 했었지 않나요?
지금은 라우팅되는 것 같아서 여쭤봅니다!

@effozen
Copy link
Collaborator Author

effozen commented Nov 26, 2024

저희 사용자 눌렀을 때, 라우팅 변하는 거 아니고 그냥 상태 변화로 하기로 했었지 않나요? 지금은 라우팅되는 것 같아서 여쭤봅니다!

맞습니다. 작업하고 올릴까하다가, GuestView도 있고, 이것까지 수정하고 올리면, PR날리는 텀이 너무 길어질 것 같아서 올렸습니다.

지금 현재 수정중입니다.

@effozen effozen closed this Nov 26, 2024
@effozen effozen deleted the feature/fe/#113-bottom-right-guide branch December 5, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants