Conversation
…ment-period-MOA-622 모집 마감 정보 노출 및 지원하기 버튼 스타일 통일
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| Cohort / File(s) | 요약 |
|---|---|
ClubApplyButton 스타일 frontend/src/pages/ClubDetailPage/components/ClubApplyButton/ClubApplyButton.styles.ts |
비활성화되지 않은 상태의 배경색을 colors.primary[800]에서 colors.gray[900]으로 변경하고 모바일 미디어 쿼리의 배경색 오버라이드 제거 |
모집 마감 텍스트 로직 frontend/src/utils/getDeadLineText.ts |
RECRUITMENT_STATUS에 새로운 LAST 상태('지난 모집') 추가. CLOSED 분기에서 recruitmentEnd 존재 여부에 따라 조건부로 마감일을 '지난 모집・M/d' 형식으로 반환하도록 변경 |
예상 코드 리뷰 규모
🎯 2 (Simple) | ⏱️ ~10 minutes
연관된 가능성 있는 PR
- 모집 마감 정보 노출 및 지원하기 버튼 스타일 통일 #1163: ApplyButton 활성화 배경색 통일 및 getDeadLineText의 LAST 상태 추가, recruitmentEnd 기반 날짜 표시 로직이 동일하게 적용됨
- [release] FE v1.1.9 #1001: 동일한 파일(getDeadLineText.ts, ClubApplyButton.styles.ts) 수정 및 모집 상태/마감일 로직 변경
- [HotFix] 동아리 상세페이지 모집 버튼 푸터 삭제 복구 #978: ClubDetailPage UI 로직 수정으로 ClubApplyButton.styles.ts 스타일 조정 및 마감일 텍스트 흐름 개선
제안 라벨
💻 FE, 📈 release
제안 리뷰어
- oesnuj
- lepitaaar
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | PR 제목이 변경사항과 부분적으로만 관련이 있습니다. 제목은 버전 릴리스를 나타내지만, 실제 변경사항(모집 마감 후 날짜 표시, 버튼 색상 변경, null 처리)의 구체적인 내용을 반영하지 않습니다. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
- 📝 Generate docstrings
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
- Commit unit tests in branch
develop-fe
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
frontend/src/pages/ClubDetailPage/components/ClubApplyButton/ClubApplyButton.styles.ts (1)
39-44:⚠️ Potential issue | 🔴 Critical모바일 미디어 쿼리 블록의 닫는
}누락 — CSS 구문 오류
${media.mobile} {블록이 열린 후 닫는}가 없이 바로 styled-component 템플릿 리터럴이 종료됩니다. 기존에background-color오버라이드를 제거하면서 닫는 중괄호도 함께 삭제된 것으로 보입니다. 이 상태로는 CSS가 깨져서 모바일 스타일이 정상 적용되지 않을 수 있습니다.🐛 수정 제안
${media.mobile} { width: 273px; height: 44px; font-size: 16px; font-weight: 500; + } `;
🧹 Nitpick comments (1)
frontend/src/utils/getDeadLineText.ts (1)
18-21: 불필요한 non-null assertion (!) 연산자Line 18에서
recruitmentEnd가null인 경우 이미 early return 하므로, Line 20의recruitmentEnd!에서!는 불필요합니다. TypeScript가 이미 narrowing으로 non-null을 추론합니다.♻️ 수정 제안
- const lastDate = format(recruitmentEnd!, 'yyyy M/d', { locale: ko }); + const lastDate = format(recruitmentEnd, 'yyyy M/d', { locale: ko });
#️⃣연관된 이슈
📝작업 내용
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
출시 노트
신기능
스타일