-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] 스케줄러 카드 추가 및 탭 컴포넌트 수정 #82
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
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
🎨 Storybook Report✨ Story가 변경되었습니다 Chromatic에서 비주얼 변경사항을 확인하세요.
|
📊 Coverage Report
📈 #82을 main에 병합하면 coverage가 Coverage 요약@@ Coverage Diff @@
## main #82 +/- ##
===========================================
+ Coverage 33.18% 36.85% +3.67%
===========================================
Files 80 80 0
Lines 2429 2491 +62
Branches 162 158 -4
===========================================
+ Hits 806 918 +112
- Misses 1623 1573 -50 영향받은 파일
|
🚀 PR Preview Report✨ Build가 성공적으로 완료되었습니다. Preview에서 변경사항을 확인하세요.
|
HopeFullee
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.
굿또 자부 나이수 자부 스바라시 자부
| }: CardParticipationRowProps) => { | ||
| return ( | ||
| <div className='mt-[14px] flex items-center'> | ||
| <div className='mt-3 flex h-[18px] items-center'> |
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.
위 훈수
| <div className='mt-3 flex h-[18px] items-center'> | |
| <div className='mt-3 flex h-4.5 items-center'> |
아래 훈수
| <CardTags tags={cardTags} /> | ||
|
|
||
| <div className='mt-3 flex flex-col gap-1'> | ||
| <div className='mt-[13px] flex flex-col gap-1'> |
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.
훈수
| <div className='mt-[13px] flex flex-col gap-1'> | |
| <div className='mt-3.25 flex flex-col gap-1'> |
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.
탭이 스크롤을 내려도 화면 상단에 고정되도록 하면 어떨까요?
팔로워가 많아져서 스크롤을 내리다가 채팅 탭으로 전환하고 싶을 때 다시 스크롤을 올려야 하는 불편함이 있을 것 같아 말씀드립니다!
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.
오 알려주셔서 감사합니다 ~~
📝 변경 사항
TabNavigation 컴포넌트 추가
basePath,paramName커스터마이징 가능Button 컴포넌트 확장
xs사이즈 추가 (h-10, 40px, radius: 12px)leavevariant 추가 (모임 탈퇴 버튼용)chatvariant 추가 (채팅 입장 버튼용)Card 컴포넌트 확장
leaveAndChatActionsprop 추가 (조건부 버튼 렌더링)스케줄러 페이지 구현
/schedule?tab=current)(components)로 탭별 컴포넌트 분리current.tsx- 현재 모임my.tsx- 나의 모임history.tsx- 모임 이력🔗 관련 이슈
Closes #67
🧪 테스트 방법
수동 테스트 검증(로컬 환경)
/schedule,/schedule?tab=current,/schedule?tab=my,/schedule?tab=history접근 확인유닛 테스트 검증
pnpm test tab-navigation --coverage통합 테스트 검증
pnpm storybook→ Shared/TabNavigation📸 스크린샷 (선택)
📋 체크리스트
관련 문서를 업데이트했습니다 (필요한 경우)
테스트를 추가/수정했습니다 (필요한 경우)
Breaking change가 있다면 명시했습니다
💬 추가 코멘트
TabNavigation 사용법 (cc. @wooktori )
기본 사용:
페이지 구현:
URL 예시:
/schedule→ 현재 모임 (디폴트)/schedule?tab=my→ 나의 모임/schedule?tab=history→ 모임 이력참고
paramName커스터마이징 가능 (기본: 'tab')CodeRabbit Review는 자동으로 실행되지 않습니다.
Review를 실행하려면 comment에 아래와 같이 작성해주세요