[Fix] 지도 페이지 및 전체적인 QA 반영 #74
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
## Walkthrough
SVG 아이콘의 경로를 `/svgs/common/` 하위 디렉토리로 일괄 변경하였고, 회원가입 및 카카오 로그인 플로우에서 `completeSignup` 메서드를 호출하여 인증 상태에 `isSignedUp` 플래그를 추가 및 갱신하도록 수정했습니다. 일부 UI 위치 및 스타일, 마커 클릭 관련 콘솔 로그, 마커 생성 로직도 소폭 조정되었습니다.
## Changes
| 파일(들) | 변경 요약 |
|---|---|
| src/pages/join/components/GoHome.jsx<br>src/pages/join/components/LocationStep.jsx<br>src/pages/join/components/OwnerStep.jsx<br>src/pages/join/components/ProfileImageStep.jsx<br>src/pages/map/MapPage.jsx<br>src/pages/map/components/IntroModal.jsx<br>src/pages/map/components/PlaceBottomSheet.jsx<br>src/pages/map/components/PlaceContent.jsx<br>src/pages/search/components/RecentSearchList.jsx<br>src/pages/search/components/SearchBar.jsx<br>src/pages/support/components/ConsumerTab.jsx | SVG 아이콘 경로를 `/svgs/`에서 `/svgs/common/`으로 변경 |
| src/pages/join/components/NameStep.jsx | SVG 경로 변경, 뒤로가기 버튼 UI 위치 및 크기 조정, 버튼 클래스명 정리(로직 변경 없음) |
| src/pages/join/SignUp.jsx | 회원가입 성공 시 `useAuthStore.getState().completeSignup()` 호출 추가 |
| src/pages/kakaoAuth/kakaoAuth.jsx | 기존 회원 로그인 시 `completeSignup()` 호출 추가 |
| src/pages/search/SearchPage.jsx | `useAuthStore` 임포트 위치 조정 및 `getLikedCompanies` 추가, 훅 위치 변경, 리팩토링 및 주석 정리, 선택 장소 좋아요 상태 재검증 로직 추가 |
| src/pages/search/hooks/useMapViewer.js | 마커 클릭 시 콘솔 로그 위치 변경, 마커 생성 초기화 로직 제거 |
| src/pages/map/hooks/useToggleLike.js | 좋아요 토글 로직 단순화, 서버 상태 기반 직접 토글 처리 및 상태 업데이트를 `requestAnimationFrame`으로 지연 |
| src/store/authStore.js | `isSignedUp` 상태 및 `completeSignup` 메서드 추가 |
## Possibly related PRs
- Morak-Team/Frontend#39: `SignUp.jsx`의 다단계 회원가입 UI 및 API 통합 구현 PR로, 본 PR과 회원가입 완료 상태 처리 관련 코드가 연결됨.
## Suggested reviewers
- dbstj0403Note ⚡️ AI Code Reviews for VS Code, Cursor, WindsurfCodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback. Note ⚡️ Faster reviews with cachingCodeRabbit now supports caching for code and dependencies, helping speed up reviews. This means quicker feedback, reduced wait times, and a smoother review experience overall. Cached data is encrypted and stored securely. This feature will be automatically enabled for all accounts on May 16th. To opt out, configure 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (4)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/pages/search/hooks/useMapViewer.js (1)
43-43: 마커 클릭 이벤트 추적을 위한 로그 추가됨마커 클릭 이벤트를 콘솔에 기록하는 코드가 추가되었습니다. 이는 "검색 후 핀 클릭 시 바텀시트가 다시 나타나지 않는 버그"를 해결하기 위한 디버깅 목적으로 보입니다.
프로덕션 코드에서는 디버깅용 콘솔 로그를 제거하는 것이 좋습니다. 문제가 해결된 후에는 이 로그를 제거하는 것을 고려해보세요.
- console.log("marker");
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (7)
public/svgs/common/Ic_All.svgis excluded by!**/*.svgpublic/svgs/common/Ic_Arrow_Left.svgis excluded by!**/*.svgpublic/svgs/common/Ic_Heart-Empty.svgis excluded by!**/*.svgpublic/svgs/common/Ic_Heart_Fill.svgis excluded by!**/*.svgpublic/svgs/common/Ic_X.svgis excluded by!**/*.svgpublic/svgs/common/Ic_X_Btn.svgis excluded by!**/*.svgpublic/svgs/common/favicon.svgis excluded by!**/*.svg
📒 Files selected for processing (17)
src/pages/join/SignUp.jsx(2 hunks)src/pages/join/components/GoHome.jsx(1 hunks)src/pages/join/components/LocationStep.jsx(1 hunks)src/pages/join/components/NameStep.jsx(3 hunks)src/pages/join/components/OwnerStep.jsx(1 hunks)src/pages/join/components/ProfileImageStep.jsx(1 hunks)src/pages/kakaoAuth/kakaoAuth.jsx(1 hunks)src/pages/map/MapPage.jsx(1 hunks)src/pages/map/components/IntroModal.jsx(1 hunks)src/pages/map/components/PlaceBottomSheet.jsx(1 hunks)src/pages/map/components/PlaceContent.jsx(2 hunks)src/pages/search/SearchPage.jsx(1 hunks)src/pages/search/components/RecentSearchList.jsx(1 hunks)src/pages/search/components/SearchBar.jsx(1 hunks)src/pages/search/hooks/useMapViewer.js(1 hunks)src/pages/support/components/ConsumerTab.jsx(1 hunks)src/store/authStore.js(1 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (2)
src/pages/join/SignUp.jsx (1)
src/store/authStore.js (1)
useAuthStore(4-25)
src/pages/kakaoAuth/kakaoAuth.jsx (1)
src/store/authStore.js (1)
useAuthStore(4-25)
🔇 Additional comments (24)
src/pages/support/components/ConsumerTab.jsx (1)
71-75: 아이콘 경로 일관성 업데이트 확인
"전체 보기" 아이콘의 경로가/svgs/common/Ic_All.svg로 변경되어 다른 컴포넌트의 SVG 구조와 일관됩니다. 빌드 및 런타임 환경에서 해당 경로에 파일이 실제로 존재하고 정상적으로 로드되는지 검증 부탁드립니다.src/pages/join/components/OwnerStep.jsx (1)
2-2: SVG 자산 import 경로 표준화 확인
BackIcon의 import 경로가/svgs/common/으로 일관되게 업데이트되어, 다른 가입 컴포넌트들과 일관성이 향상되었습니다.src/pages/join/components/ProfileImageStep.jsx (1)
2-2: SVG 자산 import 경로 표준화 확인
BackIcon의 import 경로가/svgs/common/으로 일관되게 변경되어, 다른 단계 컴포넌트들과 동일한 구조를 따릅니다.src/pages/join/components/GoHome.jsx (1)
2-2: SVG 자산 import 경로 표준화 확인
BackIcon의 import 경로가/svgs/common/디렉토리로 업데이트되어, 가입 플로우 전반에 걸쳐 일관성을 유지합니다.src/pages/map/components/IntroModal.jsx (1)
10-10: SVG 자산 import 경로 표준화 확인
닫기 아이콘의 경로가/svgs/common/으로 일관되게 변경되어, 공통 자산 관리가 용이해졌습니다.src/pages/search/components/RecentSearchList.jsx (1)
29-29: SVG 자산 import 경로 표준화 확인
삭제 아이콘의 import 경로가/svgs/common/으로 업데이트되어, SVG 파일 구조가 통일되었습니다.src/pages/map/components/PlaceBottomSheet.jsx (1)
158-162: 경로 변경이 올바르게 적용되었습니다.닫기 아이콘의 경로가
/svgs/common/Ic_X.svg로 표준화되었습니다. 이는 SVG 자산을 공통 디렉토리로 이동하는 전반적인 리팩토링의 일부로 적절한 변경입니다.src/pages/map/MapPage.jsx (3)
193-194: 하트 아이콘 경로가 올바르게 표준화되었습니다.하트 아이콘(채워진/빈)의 경로가
/svgs/common/디렉토리로 표준화되어 일관된 자산 관리가 가능해졌습니다.
177-181: 하트 아이콘 경로가 올바르게 표준화되었습니다.좋아요 상태에 따라 표시되는 하트 아이콘들의 경로가
/svgs/common/디렉토리로 표준화되어 일관된 자산 관리가 가능해졌습니다. 이는 코드베이스의 전반적인 정리 작업의 일환입니다.
160-167:Details
❌ Incorrect review comment
PR 목표에 언급된 마커 이벤트 재바인딩 관련 수정이 이 파일에 없습니다.
PR 목표에 언급된 "검색 후 핀 클릭 시 바텀시트가 다시 나타나지 않는 버그 수정"과 관련된 코드 변경이 이 파일에서는 보이지 않습니다. 마커 이벤트 재바인딩 관련 수정은
MapViewer컴포넌트나 다른 파일에 있을 수 있으니 확인이 필요합니다.
🏁 Script executed:
#!/bin/bash # MapViewer 컴포넌트에서 마커 이벤트 재바인딩 관련 코드 확인 fd MapViewer.jsx src/ -x cat {} | grep -A 10 -B 10 "marker\|핀\|바텀시트\|bottom sheet"Length of output: 1320
useMapViewer 훅에서 마커 이벤트 재바인딩 로직이 적용되었는지 확인합니다.
🏁 Script executed:
#!/bin/bash # useMapViewer hook 파일에서 이벤트 관련 코드 검색 FILES=$(fd useMapViewer src/pages/map/hooks -t f) for FILE in $FILES; do echo "==== $FILE ====" grep -nE "marker|addListener|onMarkerClick|removeListener|bottom|sheet" "$FILE" || echo "No matches in $FILE" doneLength of output: 1403
수정 불필요: useMapViewer 훅에 이미 마커 이벤트 재바인딩 로직이 구현되어 있습니다.
useMapViewer 훅 내부에서 markersRef를 관리하며
– 새로운 마커 생성 시addListener(marker, "click", …)
– 기존 마커 제거 시marker.setMap(null)및 이벤트 해제
– 매 렌더링마다 markersRef를 갱신
등의 로직이 포함되어 있어 MapViewer 컴포넌트에서 별도 수정 없이도 클릭 이벤트가 정상 재바인딩됩니다.이 리뷰는 무시해 주세요.
Likely an incorrect or invalid review comment.
src/store/authStore.js (2)
6-6: 로그인 상태와 별도로 가입 완료 상태를 추적하는 플래그가 추가되었습니다.
isSignedUp플래그를 추가하여 회원가입 완료 상태를 별도로 추적할 수 있게 되었습니다. 이는 PR 목표에 언급된 "회원가입 페이지에서 로그인 완료 처리가 제대로 되지 않는 오류"를 해결하는 첫 단계입니다.
9-9: 회원가입 완료를 처리하는 메소드가 추가되었습니다.
completeSignup메소드를 통해 회원가입 완료 상태를 명시적으로 설정할 수 있게 되었습니다. 이를 통해 회원가입과 로그인 상태를 분리하여 보다 명확한 사용자 경험 흐름 관리가 가능해졌습니다.src/pages/map/components/PlaceContent.jsx (3)
130-130: 온도 표시가 정수형으로 개선되었습니다.
.toFixed(0)를 사용하여 온도 값에서 소수점을 제거하고 정수로만 표시하도록 수정했습니다. 이는 PR 목표에 언급된 UI 개선사항과 일치하며, 사용자에게 더 깔끔한 표시를 제공합니다.
132-132: 방문자 리뷰 텍스트의 마크업 구조가 개선되었습니다.방문자 리뷰 카운트 텍스트가 별도의 span 요소로 분리되어 마크업 구조가 개선되었습니다. 이를 통해 스타일링이 더 용이해지고 텍스트 간의 관계가 더 명확해졌습니다.
177-181: 하트 아이콘 경로가 표준화되었습니다.좋아요 기능의 하트 아이콘 경로가
/svgs/common/디렉토리로 표준화되었습니다. 이는 앱 전반의 자산 관리 개선 작업의 일부로, 일관성을 높이고 유지보수성을 개선합니다.src/pages/search/components/SearchBar.jsx (1)
9-9: SVG 경로 업데이트가 완료되었습니다.SVG 아이콘 경로가
/svgs/common/디렉토리로 통일되어 파일 구조가 더 체계적으로 정리되었습니다.src/pages/join/components/LocationStep.jsx (1)
2-3: SVG 아이콘 경로가 적절히 업데이트되었습니다.XIcon과 BackIcon의 경로가
/svgs/common/디렉토리로 변경되어 PR 목표에 맞게 파일 구조가 표준화되었습니다.src/pages/join/SignUp.jsx (2)
11-11: 필요한 스토어 모듈을 정확히 가져왔습니다.회원가입 완료 상태 관리를 위해 필요한
useAuthStore가 적절히 임포트되었습니다.
41-41: 회원가입 완료 상태 처리가 적절히 구현되었습니다.회원가입 API 호출 성공 후
completeSignup()메서드를 호출하여isSignedUp상태를 업데이트하도록 수정되었습니다. 이 변경으로 PR에서 언급한 "회원가입 시 로그인 완료 처리가 제대로 되지 않는 문제"가 해결되었습니다.src/pages/kakaoAuth/kakaoAuth.jsx (1)
27-27: 로그인 완료 후 회원가입 상태 업데이트 추가기존 사용자에 대해
completeSignup()메서드를 호출하여 로그인 성공 시 회원가입 상태를 업데이트하는 코드가 추가되었습니다. 이는 PR 목표에 언급된 "회원가입 페이지에서 로그인 완료 처리가 제대로 되지 않는 오류"를 해결하기 위한 변경사항입니다.이 변경은
useAuthStore의isSignedUp상태 플래그를 업데이트하여 로그인 및 회원가입 흐름을 일관되게 관리할 수 있게 합니다.src/pages/join/components/NameStep.jsx (3)
2-3: 아이콘 경로 표준화SVG 아이콘 경로가
/svgs/common/디렉토리로 표준화되었습니다. 이는 PR 목표에 언급된 "공용 파일 정리" 작업의 일환으로 보입니다.
24-28: 뒤로가기 버튼 위치 표준화뒤로가기 버튼의 위치가 상단 중앙에 고정되도록 변경되었습니다. 이전에는 절대 위치로 배치되었을 것으로 보이나, 이제 화면 너비에 따라 자동으로 중앙 정렬되며 더 일관된 UI를 제공합니다.
이 변경은 PR 목표에 언급된 "회원가입 단계의 뒤로가기 버튼 위치 표준화"를 직접적으로 구현한 것입니다.
80-80: 조건부 클래스 표현식 단순화조건부 클래스 표현식이 단순화되었습니다. 기능적인 변화는 없지만 코드가 더 간결해졌습니다.
src/pages/search/SearchPage.jsx (1)
229-229: 마커 이벤트 다시 바인딩을 위한 개선검색 결과 마커에
isSearchResult: true플래그를 추가하여 구분할 수 있게 했습니다. 이는 PR 목표에 언급된 "검색 페이지에서 핀 클릭 후 하단 시트가 다시 나타나지 않는 버그" 수정과 관련됩니다.이 변경으로 마커 클릭 시 검색 결과임을 식별하여 하단 시트가 적절히 표시될 수 있습니다. 마커 이벤트가 다시 바인딩되어 사용자 경험이 개선됩니다.
|
🌟 어떤 이유로 MR를 하셨나요?
📝 세부 내용 - 왜 해당 MR이 필요한지 작업 내용을 자세하게 설명해주세요
검색 페이지 관련 버그 수정
toFixed(0))회원가입 페이지 관련 버그 수정
isSignedUp상태 분리)public 파일 정리
📸 작업 화면 스크린샷
📢 로컬 실행 시 유의사항
isSaved) 와 클라이언트 상태 불일치 이슈 해결 기다리고 있습니다.🚨 관련 이슈 번호
Summary by CodeRabbit
신규 기능
버그 수정
스타일
/svgs/common/디렉토리로 통일되었습니다.기타