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

[18기 송나은] StoreDetail 레이아웃 완료 (Map, Roadview API 포함) #5

Merged
merged 1 commit into from Apr 5, 2021

Conversation

songbetter
Copy link
Contributor

@songbetter songbetter commented Mar 31, 2021

수정 사항 간략한 한줄 요약

  • StoreDetail 레이아웃 완료 (Map, Roadview API 포함)

수정 사항들 자세한 내용

  • StoreDetail 레이아웃 완료
  • Map, Roadview 카카오지도 API 불러오기 완료
  • 데이터 패치 Test 완료
  • Menu 더보기 기능 추가 구현 (이미지 4개 -> 이미지 전체)
  • 메뉴판 추가 구현
  • 방문자사진, 방문자리뷰 추가 구현
  • API 데이터 통신 완료

기타 질문 및 특이 사항

  • 토글 버그가 있습니다.
  • 스토어리스트와 합쳐보고 수정되지 않은 버그는 추후 수정하겠습니다.!

체크 리스트 (아래 사항들이 전부 체크되어야만 merge가 됩니다!)

  • 필요한 test들을 완료하였고 기능이 제대로 실행되는지 확인 하였습니다.
  • Wecode의 코드 스타일 가이드에 맞추어 코드를 작성 하였습니다.
  • 제가 의도한 파일들과 수정 사항들만 커밋이 된 것을 확인 하였습니다.
  • 본 수정 사항들을 팀원들과 사전에 상의하였고 팀원들 모두 해당 PR에 대하여 알고 있습니다.
  • Git rebase와 squash를 했고 커밋 수가 하나 인것을 확인 했습니다.

Copy link
Contributor

@bonbon0202 bonbon0202 left a comment

Choose a reason for hiding this comment

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

나은님! 수고하셨습니다 :)
카카오 지도 잘 셋팅해주셨네요.
더불어 hooks, styled-components도 잘 사용해주셨는데, 반복되는 코드는 더 간결화할 수 없는지 고민해주시고, sass사용과 styled components사용이 비효율적으로 혼용되고 있지 않은지 전체적으로 확인 후 수정해주세요!

src/Pages/Main/Components/StoreDetail/Menu.js Outdated Show resolved Hide resolved
src/Pages/Main/Components/StoreDetail/Menu.js Outdated Show resolved Hide resolved
src/Pages/Main/Components/StoreDetail/Roadview.js Outdated Show resolved Hide resolved
src/Pages/Main/Components/StoreDetail/Roadview.js Outdated Show resolved Hide resolved
src/Pages/Main/Components/StoreDetail/StoreDetail.js Outdated Show resolved Hide resolved
Copy link
Contributor

@bonbon0202 bonbon0202 left a comment

Choose a reason for hiding this comment

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

나은님! 수고하셨어요!!
슬라이드 구현! ;) 굳굳! 잘하셨습니다.
오늘 세션 했던 '함수재사용'부분도 고민하시면서 재사용 가능한 부분은 하나로 합쳐 사용해보세요!

src/Pages/Main/Components/StoreDetail/ImageSlider.js Outdated Show resolved Hide resolved
src/Pages/Main/Components/StoreDetail/ImageSlider.js Outdated Show resolved Hide resolved
src/Pages/Main/Components/StoreDetail/Info.js Outdated Show resolved Hide resolved
src/Pages/Main/Components/StoreDetail/ImageSlider.js Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants