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

FEAT: Map 관련 페이지 API 연동 #36

Merged
merged 21 commits into from
Jan 17, 2024
Merged

FEAT: Map 관련 페이지 API 연동 #36

merged 21 commits into from
Jan 17, 2024

Conversation

chj950807
Copy link
Collaborator

@chj950807 chj950807 commented Jan 14, 2024

작업한 내용

  • Map 페이지 API 연동 및 관련 type 수정

  • SearchLocation 페이지 API 연동 및 관련 type 수정

  • Branch 페이지 API 연동 및 관련 type 수정

  • type 수정에 따라 불필요한 함수 제거

  • 디자인 요청 사항 수정

  • 주변에 branch가 없을경우 Toast UI 추가


리뷰 시 참고사항

  • Map에서 같은 좌표에 Marker가 여러개 찍히면 Marker가 보이지 않는 문제가 있음.
  • BranchCard, PhotoDump 등에서 해당하는 데이터들이 없다면 빈 배열이 아닌 아무것도 안보내주는 문제가 있어 수정 요청한 상태입니다.
  • @env 에서 import 해 올시 에러 라인 발생, babel.config 문제로 추정되는데 해결 못하고 넘어간 상태입니다. 방법 아신다면 도와주시면 감사하겠습니다.
  • API 에러 처리 관련해서는 Backend 에서 수정 완료되면 처리 로직 구성 예정입니다.

PR type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other (설명):

@sunujun

This comment was marked as resolved.

@sunujun sunujun merged commit 8a330b2 into main Jan 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants