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][Refactor] #224 : 지도 컴포넌트 리팩토링 #247

Merged
merged 1 commit into from
Nov 24, 2024
Merged

[FE][Refactor] #224 : 지도 컴포넌트 리팩토링 #247

merged 1 commit into from
Nov 24, 2024

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 23, 2024

📝 PR 개요

지도 컴포넌트 리팩토링

🔍 변경 사항

통합 과정에서 지나치게 스파게티 코드가 되어, 손쓰기 어려울 듯 해서 지도 컴퍼넌트 수정

✅ 체크리스트 (Checklist)

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

🔄 관련 이슈 (Linked Issues)

#224

@effozen
[FE][Refactor] #224 : 지도 컴포넌트 리팩토링
df7cc79 
- 코드 가독성 개선
- 에러 케이스 추가
- 타입 분리
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 23, 2024
leedongyull
leedongyull approved these changes Nov 24, 2024
View reviewed changes
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 2f32b00 into frontend Nov 24, 2024
3 of 4 checks passed
@effozen effozen deleted the feature/fe/#224-combine-map-and-canvas 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

@juwon5272 juwon5272 juwon5272 approved these changes

@leedongyull leedongyull leedongyull approved these changes

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 @leedongyull @juwon5272