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][Fix][Refactor] #112 : 헤더 컴포넌트 외부에서 인자 받을 수 있는 기능 추가 및 버그 수정 #178

Merged
merged 9 commits into from
Nov 17, 2024

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 16, 2024

📝 PR 개요

공통 헤더 컴포넌트 수정

🔍 변경 사항

  • 헤더 컴포넌트 외부에서 dropdown 아이템 받을 수 있게 기능 추가
  • Dropdown 아이템 별로 아이콘 색상 변경 안되는 문제 수정
  • 조금 더 범용성 있게 헤더 리팩토링

✅ 체크리스트 (Checklist)

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

🔄 관련 이슈 (Linked Issues)

#112

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

스크린샷 2024-11-17 오전 5 00 38

🧪 테스트 방법

추후 문서화와 함께 추가 예정

📚 참고 자료 (선택 사항)

happyhyep and others added 9 commits November 14, 2024 18:14
- docker-compose 파일 작성
- nginx 파일 작성
- 백엔드 dockerfile 작성
- local 테스트 완료
…ackend

[BE][Chore] #166 : docker-compose, nginx 설정 및 백엔드 dockerfile 작성
[FE][Feat] : 배포를 위한 머지
- pnpm-lock.yaml 파일 수정
[BE][Chore] #166 : docker-compose, nginx 설정 및 백엔드 dockerfile 작성
- jenkinsfile 파일 작성
[BE][Chore] #166 : jenkinsfile 세팅
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 16, 2024
@effozen effozen changed the base branch from main to frontend November 16, 2024 20:06
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.

리팩토링할 부분은 ... 약속했듯 추후에 남겨두겠습니다!! 고생하셨습니다!

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.

LGTM

@effozen effozen merged commit f455eaa into frontend Nov 17, 2024
3 of 6 checks passed
@effozen
Copy link
Collaborator Author

effozen commented Nov 17, 2024

리팩토링할 부분은 ... 약속했듯 추후에 남겨두겠습니다!! 고생하셨습니다!

나중에 이슈로 달아주세요! 우선은 시간 크게 안걸리는것만 빠르게 수정했고, 인자같은 경우는 Dropdown 등에 서버에서 받은 정보를 출력해야할 거 같아서 추가해두었습니다!

심각한 이슈가 아니라면, 리팩토링에 대해서는 몰아서 개발 끝나고 리팩토링 주간이나 이럴 때 해보고자 합니다.

금일 통신로직 및 지도-캔버스 연동 작업 들어가겠습니다.

@effozen effozen added 작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 17, 2024
@effozen effozen deleted the feature/fe/#112-component-refactoring branch November 17, 2024 06:24
@github-actions github-actions bot added 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
작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants