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][Feat] #109 : 손녀 시점에서의 화면 구현 #163

Closed
wants to merge 4 commits into from

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 14, 2024

📝 PR 개요

  • 지도위에 Header 컴포넌트 겹쳐서 보여주는 기능 구현

🔍 변경 사항

  • 지도위에 헤더 컴포넌트 구현

✅ 체크리스트 (Checklist)

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

🔄 관련 이슈 (Linked Issues)

close #109

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

스크린샷 2024-11-14 오전 10 01 58

🧪 테스트 방법

📚 참고 자료 (선택 사항)

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 14, 2024
onClick: React.MouseEventHandler<HTMLButtonElement>;
}

// TODO: 이렇게 쓸 거면 범용 버튼으로 빼도...?
Copy link
Collaborator

Choose a reason for hiding this comment

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

BackButton이라기엔 범용적으로 보이기는 하네요ㅎㅎ 뒤로가기 기능이 여기에 있는거도 아니라 추후에 빼도 좋을거 같습니다

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.

이전 PR인 [FE][Feat} #110 : 뒤로가기 버튼 구현와 같은 코드가 올라간 것 같아보여요...!!
브랜치는 다른 브랜치로 올리셨는데, 동일한 내용을 올리신 것 같아서,,, 브랜치 별로 해당 기능만 구현해주시고, 그 내용만 PR로 올라갈 수 있게 해주시면 좋을 것 같습니다!
이전 PR에 코멘트 남기고 request changes 해뒀으니 우선 이 PR에는 approve 해둘게요!

@effozen
Copy link
Collaborator Author

effozen commented Nov 14, 2024

크게 손볼게 있어서 다시 PR날리겠습니다!

@effozen effozen closed this Nov 14, 2024
@effozen effozen added 작업 중지 기술 구현 한계, 우선 순위 등 작업을 중지해야 하는 어쩔 수 없는 이유로 중지된 상태 and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 14, 2024
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 14, 2024
@github-actions github-actions bot requested a review from juwon5272 November 14, 2024 05:21
@effozen effozen deleted the feature/fe/#109-show-component-on-map branch November 16, 2024 21:07
@effozen effozen removed the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
작업 중지 기술 구현 한계, 우선 순위 등 작업을 중지해야 하는 어쩔 수 없는 이유로 중지된 상태
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants