[REFACTOR][UHYU-434] 마이페이지 및 관리자 페이지 리드미 보완 #238
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Contributor
|
Caution Review failedThe pull request is closed. WalkthroughREADME.md의 "마이페이지" 및 "관리자(Admin)" 섹션의 설명이 구체적으로 보완되고, 기능 항목이 세분화되었습니다. UI/UX 관련 설명이 추가되었으며, 기존 이미지는 보다 상세한 스크린샷으로 교체 또는 재배치되었습니다. 코드나 엔티티 선언의 변경은 없습니다. Changes
Sequence Diagram(s)(해당 변경은 문서 보완 및 파일 삭제에 한정되어 있어 시퀀스 다이어그램 생성이 필요하지 않습니다.) Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changes(연관 이슈의 목적을 벗어난 변경사항은 발견되지 않았습니다.) Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests
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. 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}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
README.md (3)
145-148: 긴 항목은 소제목 / 서브리스트로 분리하여 가독성 개선 제안
세 개의 기능 설명이 한 줄에 길게 나열돼 있어 한눈에 파악하기 어렵습니다. 예를 들어 “개인별 활동내역 확인” 이후 괄호 안 세부 항목을 서브 리스트로 분리하면 읽기 편해집니다.-- 개인별 활동내역 확인 (총 멤버십 사용 혜택 금액, 사용자 클릭 패턴 관심사 결과, 최근 방문 기록) +- 개인별 활동내역 확인 + - 총 멤버십 사용 혜택 금액 + - 사용자 클릭 패턴 관심사 결과 + - 최근 방문 기록
167-176: 중복 / 유사 표현 정리 필요
“저장된 매장 통계 상세 조회 기능”과 “카테고리별 브랜드 상세 조회” 등이 상위 bullet “U-HYU 전체 통계 조회 기능”과 구조적으로 동등한 항목인지, 하위 세부 기능인지 판단이 어려워 계층 혼란이 있습니다. 중첩 bullet 구조를 사용해 상-하위 관계를 명확히 해 주세요.
182-185: 실시간 조회 기능 위치 조정 권장
“검색 및 필터링으로 인한 실시간 조회”는 CRUD 기능과 동일 단계 bullet 로 나열돼 있어 기능 그룹핑이 모호합니다. ‘조회’가 List View 기능과 겹치므로 CRUD 안의 ‘R(Read)’로 포함하거나 서브 리스트로 이동하는 편이 자연스럽습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(2 hunks)
🧰 Additional context used
🧠 Learnings (5)
📓 Common learnings
Learnt from: pillow12360
PR: U-Final/U-Hyu-fe#85
File: src/features/admin/hooks/useAdminBrandQuery.ts:51-52
Timestamp: 2025-07-27T07:59:08.617Z
Learning: 관리자 페이지에서 로딩 상태 개선을 위한 스켈레톤 UI 도입이 계획되어 있음. 차트 데이터 로딩과 브랜드 CRUD 작업 시 사용자 경험 향상을 목적으로 함.
Learnt from: pillow12360
PR: U-Final/U-Hyu-fe#0
File: :0-0
Timestamp: 2025-07-25T00:11:22.930Z
Learning: 관리자 페이지의 보호 라우트 처리 및 효율적인 로그인 여부 판단을 위한 리펙토링과 기능 설계가 예정되어 있음. 현재는 useKakaoLogin 훅과 BottomSheetContainer에서의 기본적인 로그인 확인 로직이 구현된 상태.
Learnt from: pillow12360
PR: U-Final/U-Hyu-fe#0
File: :0-0
Timestamp: 2025-07-24T00:36:55.626Z
Learning: PR #71에서 PersistentBottomSheet를 MapDragBottomSheet로 교체하면서 BottomSheetContainer의 구조가 steps 배열에서 getCurrentStepContent() 함수 기반으로 변경됨. 각 스텝의 헤더 UI가 컴포넌트 내부로 이동하고, 3단계 드래그 시스템(collapsed/middle/expanded)이 도입됨.
📚 Learning: api 명세 변경 후 타입 개선 작업 시 고려사항: brandgrid 컴포넌트의 selectedbrands 타입을 number[] | string[]에서 number[]로 단순화 ...
Learnt from: pillow12360
PR: U-Final/U-Hyu-fe#0
File: :0-0
Timestamp: 2025-07-24T00:31:53.032Z
Learning: API 명세 변경 후 타입 개선 작업 시 고려사항: BrandGrid 컴포넌트의 selectedBrands 타입을 number[] | string[]에서 number[]로 단순화 가능하며, UserInfo 타입을 src/features/user/api/types.ts와 src/features/mypage/api/types.ts에서 통합 가능함
Applied to files:
README.md
📚 Learning: 관리자 페이지에서 로딩 상태 개선을 위한 스켈레톤 ui 도입이 계획되어 있음. 차트 데이터 로딩과 브랜드 crud 작업 시 사용자 경험 향상을 목적으로 함....
Learnt from: pillow12360
PR: U-Final/U-Hyu-fe#85
File: src/features/admin/hooks/useAdminBrandQuery.ts:51-52
Timestamp: 2025-07-27T07:59:08.617Z
Learning: 관리자 페이지에서 로딩 상태 개선을 위한 스켈레톤 UI 도입이 계획되어 있음. 차트 데이터 로딩과 브랜드 CRUD 작업 시 사용자 경험 향상을 목적으로 함.
Applied to files:
README.md
📚 Learning: 관리자 페이지의 보호 라우트 처리 및 효율적인 로그인 여부 판단을 위한 리펙토링과 기능 설계가 예정되어 있음. 현재는 usekakaologin 훅과 bottomsheetcontai...
Learnt from: pillow12360
PR: U-Final/U-Hyu-fe#0
File: :0-0
Timestamp: 2025-07-25T00:11:22.930Z
Learning: 관리자 페이지의 보호 라우트 처리 및 효율적인 로그인 여부 판단을 위한 리펙토링과 기능 설계가 예정되어 있음. 현재는 useKakaoLogin 훅과 BottomSheetContainer에서의 기본적인 로그인 확인 로직이 구현된 상태.
Applied to files:
README.md
📚 Learning: userinfo 타입이 src/features/user/api/types.ts와 src/features/mypage/api/types.ts에서 서로 다르게 정의되어 있음. user...
Learnt from: pillow12360
PR: U-Final/U-Hyu-fe#0
File: :0-0
Timestamp: 2025-07-23T11:44:09.071Z
Learning: UserInfo 타입이 src/features/user/api/types.ts와 src/features/mypage/api/types.ts에서 서로 다르게 정의되어 있음. user store에서는 age: number | null, favoriteBrands: string[]를 사용하고, mypage에서는 age: string, brandIds: number[]를 사용함
Applied to files:
README.md
🔇 Additional comments (1)
README.md (1)
140-142: 멤버십 등급 수정 가능 여부를 명시해 주세요
해당 bullet (“멤버십 등급 및 관심 브랜드 선택/해제”)가 실제로 사용자가 등급 자체를 변경할 수 있다는 의미인지, 등급에 따른 정보만 보여 준다는 의미인지 모호합니다. 기능 한계를 명확히 기술하지 않으면 오해를 불러올 수 있습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
주요 작업 내용 (전체 요약)
마이페이지 리드미 보완 (+ 즐겨찾기 이미지 추가)
관리자 페이지 리드미 보완
Summary by CodeRabbit