-
Notifications
You must be signed in to change notification settings - Fork 1
마이-메인페이지 UI 수정, 마이페이지 사이드바 수정 #238
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
Conversation
📝 WalkthroughWalkthrough이번 변경에서는 Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant MyPageMain
participant MypageProfileHeader
User->>MyPageMain: 마이페이지 메인 진입
MyPageMain->>MypageProfileHeader: name, email, profileImageUrl, onLogoutClick 전달
MypageProfileHeader-->>User: 이름과 이메일, 프로필 이미지, 로그아웃 버튼 렌더링
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
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.yaml 📒 Files selected for processing (5)
💤 Files with no reviewable changes (1)
🧰 Additional context used🧠 Learnings (12)📓 Common learnings📚 Learning: the mainpage component in apps/what-today/src/pages/main/index.tsx contains placeholder/example code...Applied to files:
📚 Learning: taeil08 prefers using translate-x-1/2 and translate-x-1/2 for carousel navigation button positioning...Applied to files:
📚 Learning: kjhyun0830은 what-today 프로젝트의 마이페이지(apps/what-today/src/pages/mypage/main/index.tsx)에서 다음 브랜치에 스켈레톤 u...Applied to files:
📚 Learning: packages/design-system/src/components/datepicker.tsx에서 taeil08은 sr-only 클래스를 가진 div 요소가 달력 트리거 버튼의 세...Applied to files:
📚 Learning: harryseop encountered a scroll restoration issue in the usebodyscrolllock hook where the base page j...Applied to files:
📚 Learning: in packages/design-system/src/components/button/index.tsx, harryseop uses handleclick wrapper for se...Applied to files:
📚 Learning: popovertrigger component in controlled mode (when open and onopenchange props are provided to popove...Applied to files:
📚 Learning: user kjhyun0830 follows an incremental development approach in what-today project, implementing ui f...Applied to files:
📚 Learning: kjhyun0830 프로젝트에서 @what-today/design-system 패키지의 reservationstatus 타입이 calendar/index.ts에서 re-export...Applied to files:
📚 Learning: taeil08이 what-today 프로젝트의 체험 등록 페이지(apps/what-today/src/pages/experiences/iindex.tsx)에서 주소 입력 컴포넌트가 ...Applied to files:
📚 Learning: myungjiwoo plans to implement form validation using zod and react-hook-form in a future iteration ra...Applied to files:
🧬 Code Graph Analysis (1)apps/what-today/src/pages/mypage/main/index.tsx (2)
🔇 Additional comments (9)
✨ 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. 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
Documentation and Community
|
|
🚀 오늘뭐해 Preview Deploy 완료! |
MyungJiwoo
left a comment
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.
이메일도 넣어주시고, 모바일에서 상태 요약 부분도 두 줄로 처리되어 가독성이 더 좋아진 것 같아요!
제가 이어서 마저 리팩토링 끝내보겠습니다!
고생하셨어요~! 👍🏻👍🏻👍🏻
🧩 관련 이슈 번호
📌 작업 내용
✅ 체크리스트
📷 UI 변경 사항 (선택)
2025-08-04.1.20.21.mov
2025-08-04.1.31.25.mov
❓무슨 문제가 발생했나요? (선택)
💬 기타 참고 사항 (선택)
Summary by CodeRabbit
신규 기능
스타일
리팩터