[fix] 소개할게요 필드 introDescription 으로 변경#976
Conversation
|
Warning
|
| 코호트 / 파일 | 변경 요약 |
|---|---|
Prop 이름 변경 frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.tsx |
Props 인터페이스에서 activityDescription: string → introDescription: string으로 이름 변경 및 컴포넌트 내부에서 사용 업데이트 |
데이터 전달 업데이트 frontend/src/pages/ClubDetailPage/ClubDetailPage.tsx |
ClubProfileCard에 전달하는 prop명을 activityDescription에서 introDescription으로 변경하고 데이터 소스를 clubDetail.description.introDescription으로 업데이트 |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related issues
- [fix] 소개할게요, 이런활동을해요 같은 필드 공유하는 버그 #974: 설명 필드(introDescription/activityDescription)를 다루는 버그로, 이 PR의 prop 이름 변경과 직접적으로 관련됨
Possibly related PRs
- [Hotfix] 소개할게요 필드 introDescription 으로 변경 #975: 동일한 ClubProfileCard prop 이름 변경(
activityDescription→introDescription)과 ClubDetailPage 데이터 소스 업데이트를 수행하는 PR
Suggested reviewers
- seongwon030
- oesnuj
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 제목이 주요 변경 사항을 정확하게 반영하고 있습니다. activityDescription을 introDescription으로 변경한 내용을 명확히 설명하고 있습니다. |
| 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/introduce-activity-change-field
📜 Recent review details
Configuration used: Organization 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 (2)
frontend/src/pages/ClubDetailPage/ClubDetailPage.tsxfrontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.tsx
🧰 Additional context used
📓 Path-based instructions (3)
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/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.tsxfrontend/src/pages/ClubDetailPage/ClubDetailPage.tsx
frontend/**/*.{tsx,jsx}
📄 CodeRabbit inference engine (frontend/.cursorrules)
frontend/**/*.{tsx,jsx}: Abstract complex logic/interactions into dedicated components/HOCs
Separate significantly different conditional UI/logic into distinct components
Colocate simple, localized logic or use inline definitions to reduce context switching
Choose field-level or form-level cohesion based on form requirements when using form libraries like react-hook-form
Use Component Composition instead of Props Drilling to reduce coupling
Files:
frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.tsxfrontend/src/pages/ClubDetailPage/ClubDetailPage.tsx
frontend/**/*.{ts,tsx}
📄 CodeRabbit inference engine (frontend/.cursorrules)
Use consistent return types for similar functions/hooks
Files:
frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.tsxfrontend/src/pages/ClubDetailPage/ClubDetailPage.tsx
🧠 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')는 로그인 기능 구현 후 동적으로 가져오는 방식으로 수정될 예정입니다.
Learnt from: seongwon030
Repo: Moadong/moadong PR: 548
File: frontend/src/pages/ClubDetailPage/ClubDetailPage.tsx:17-57
Timestamp: 2025-07-19T05:05:10.196Z
Learning: ClubDetailPage.tsx에서 notJoinedClubNames 배열의 하드코딩은 의도적인 설계 결정입니다. 개발자가 명시적으로 하드코딩을 선택했으므로 이에 대한 리팩토링 제안을 하지 않아야 합니다.
🔇 Additional comments (2)
frontend/src/pages/ClubDetailPage/ClubDetailPage.tsx (1)
45-45: 데이터 소스 변경이 올바르게 적용되었습니다.ClubProfileCard 컴포넌트에 전달하는 prop이
introDescription으로 변경되었고, 데이터 소스도clubDetail.description.introDescription으로 일관되게 업데이트되었습니다. 백엔드 API 응답의DetailedDescription타입에introDescription필드가 정의되어 있으며, ClubProfileCard 컴포넌트의 Props 인터페이스에서도 타입이 올바르게 정의되어 있습니다.frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.tsx (1)
17-17: 필드명 변경이 ClubProfileCard 컴포넌트 내에서 일관되게 적용되었습니다.Props 인터페이스(17줄), 컴포넌트 매개변수(26줄), 그리고 렌더링 부분(113줄)에서
introDescription변경이 일관되게 적용되었고, ClubDetailPage.tsx에서도 올바르게introDescription={clubDetail.description.introDescription}으로 전달되고 있습니다.다만 주의할 점은, 기존
activityDescription필드가 타입 정의(DetailedDescription)와 ClubIntroContent, ClubIntroTab 등 다른 컴포넌트에서 여전히 사용 중입니다. 이는 ClubProfileCard 컴포넌트의 로컬 prop 이름 변경이므로 breaking change 자체는 아니지만, 코드베이스 전체에서 일관된 필드명 사용을 고려했을 때 추후 리팩토링이 필요할 수 있습니다.
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.
2 participants
#️⃣연관된 이슈
#974
📝작업 내용
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
변경사항
✏️ Tip: You can customize this high-level summary in your review settings.