-
Notifications
You must be signed in to change notification settings - Fork 2
Update README.md #237
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
Update README.md #237
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughREADME.md 파일의 협업 가이드라인에 "QA" 섹션이 새로 추가되었습니다. 이 섹션에는 "MyMap 기능 리팩토링" 위키 페이지로 연결되는 단일 항목이 포함되어 있습니다. 그 외의 내용이나 구조 변경은 없습니다. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 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. ✨ 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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
README.md (1)
228-230: QA 섹션에 간단한 설명 문구 추가 권장
## QA바로 아래에 항목이 한 개만 있고 설명이 없어 처음 읽는 사람이 “QA가 무엇을 다루는지” 파악하기 어렵습니다.
헤더 아래에 한 줄짜리 설명(예: “테스트 시 확인해야 할 이슈와 개선 과정을 기록합니다.”)을 넣으면 가독성과 의도 전달이 좋아질 것 같습니다.## QA +- 서비스 품질 개선 관련 문서와 테스트 체크리스트를 모아둔 섹션입니다. - [MyMap 기능 리팩토링](https://github.com/U-Final/U-Hyu-fe/wiki/MyMap-%EA%B8%B0%EB%8A%A5-%EB%A6%AC%ED%8C%A9%ED%86%A0%EB%A7%81)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
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)이 도입됨.
주요 작업 내용 (전체 요약)
현재 UI 캡처
체크리스트
npm run fix를 실행했나요?npm run test:coverage를 실행했나요?Summary by CodeRabbit