[fix] 소개 페이지 동아리 카드의 모집 상태가 불러와지지 않는 오류를 수정한다#899
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| 응집도 / 파일 | 변경 요약 |
|---|---|
모의 데이터 업데이트 frontend/src/pages/IntroducePage/constants/mockData.ts |
플로팅 클럽 데이터셋의 모바이(moaboza) 클럽의 recruitmentStatus를 UPCOMING에서 ALWAYS로 변경 |
예상 코드 리뷰 난이도
🎯 1 (매우 간단함) | ⏱️ ~3분
관련 이슈
- [fix] 소개 페이지 동아리 카드의 모집 상태가 불러와지지 않는 오류를 수정한다 #897: IntroducePage 클럽 데이터의 recruitmentStatus 값을 다루고 있어 본 PR과 연관됨
관련 PR
- [feature] 관리자 페이지 모집기간 선택에 상시모집 처리 버튼을 추가한다 #774: ALWAYS 채용 상태 처리(UI 토글, 먼 미래 종료 날짜, 표시 로직)를 구현하며 이 PR에서 설정한 상태값을 소비함
- [feature] 동아리 상시모집 상태 추가 #707: 클럽의 recruitmentStatus를 "ALWAYS"로 변경하며, ClubStateBox에서 "ALWAYS" 상태의 UI 스타일링을 추가하는 PR과 직접 연관됨
제안 레이블
🐞 Bug
제안 검토자
- Zepelown
- oesnuj
- seongwon030
Pre-merge checks and finishing touches
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | PR 제목이 주요 변경사항과 일치합니다. 소개 페이지의 동아리 카드 모집 상태 오류 수정이라는 제목이 mockData.ts에서 moaboza 클럽의 recruitmentStatus를 UPCOMING에서 ALWAYS로 변경한 실제 변경사항을 정확히 반영합니다. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✨ 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
fix/club-card-recruit-status
Tip
📝 Customizable high-level summaries are now available in beta!
You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
- Provide your own instructions using the
high_level_summary_instructionssetting. - Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
- Use
high_level_summary_in_walkthroughto move the summary from the description to the walkthrough section.
Example instruction:
"Divide the high-level summary into five sections:
- 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
- 📓 References — List relevant issues, discussions, documentation, or related PRs.
- 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
- 📊 Contributor Summary — Include a Markdown table showing contributions:
| Contributor | Lines Added | Lines Removed | Files Changed |- ✔️ Additional Notes — Add any extra reviewer context.
Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."
Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
frontend/src/pages/IntroducePage/constants/mockData.ts (1)
20-20: 상태 값 변경은 PR 목적에 부합합니다만, 상태 문자열은 상수/타입으로 묶는 것을 고려해보세요.
recruitmentStatus: 'ALWAYS'로 바꾼 것은 소개 카드에서 "상시모집"이 제대로 노출되도록 하는 목적에 잘 맞고, 주변의'OPEN','CLOSED'와도 일관적입니다.다만 이번 이슈처럼 존재하지 않는 상태 문자열을 쓰면 런타임에서만 드러나는 버그가 다시 생길 수 있으니,
Club['recruitmentStatus']를 문자열 리터럴 유니온/enum이나 공통 상수(예:RECRUIT_STATUS.ALWAYS)로 강하게 제한하고, 목업 데이터도 그 상수를 사용하도록 정리해 두면 안전성이 더 높아질 것 같습니다. (코딩 가이드라인의 "상수 정의로 묵시적 실패 방지" 항목과도 맞습니다.)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Jira integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
frontend/src/pages/IntroducePage/constants/mockData.ts(1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
frontend/**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (frontend/.cursorrules)
frontend/**/*.{ts,tsx,js,jsx}: Replace magic numbers with named constants for clarity
Replace complex/nested ternaries withif/elseor IIFEs for readability
Assign complex boolean conditions to named variables for explicit meaning
Avoid hidden side effects; functions should only perform actions implied by their signature (Single Responsibility Principle)
Use unique and descriptive names for custom wrappers/functions to avoid ambiguity
Define constants near related logic or ensure names link them clearly to avoid silent failures
Break down broad state management into smaller, focused hooks/contexts to reduce coupling
Files:
frontend/src/pages/IntroducePage/constants/mockData.ts
frontend/**/*.{ts,tsx}
📄 CodeRabbit inference engine (frontend/.cursorrules)
Use consistent return types for similar functions/hooks
Files:
frontend/src/pages/IntroducePage/constants/mockData.ts
🧠 Learnings (1)
📓 Common learnings
Learnt from: seongwon030
Repo: Moadong/moadong PR: 195
File: frontend/src/pages/AdminPage/AdminPage.tsx:7-7
Timestamp: 2025-03-19T05:18:07.818Z
Learning: AdminPage.tsx에서 현재 하드코딩된 클럽 ID('67d2e3b9b15c136c6acbf20b')는 로그인 기능 구현 후 동적으로 가져오는 방식으로 수정될 예정입니다.
#️⃣연관된 이슈
📝작업 내용
=> 'UPCOMING' 상태가 없는데, 잘못 불러오고 있어서, 변경함
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
릴리스 노트
✏️ Tip: You can customize this high-level summary in your review settings.