[feature] ClubDetailPage에서 공유 버튼 표시 활성화#1119
Conversation
- LegacyClubDetailPage에서만 공유 버튼 숨김 조건을 유지하고, ClubDetailPage에서는 모든 기기에서 공유 버튼이 표시되도록 변경
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
suhyun113
left a comment
There was a problem hiding this comment.
빠르게 수정해주셨네요! 수고하셨습니다!!
궁금한게 있는데 새로운 ClubDetail 페이지에서 모바일 웹, 앱 모두 공유 버튼을 클릭했을때 share api를 사용하는줄 알았는데 단순 복사만 되는건 의도하신 건가요?
|
Warning
|
| Cohort / File(s) | Summary |
|---|---|
구성 설정 .gitignore |
Claude 관련 파일(CLAUDE.md, .claude)을 무시 목록에 추가 |
Club 상세 페이지 주요 컴포넌트 frontend/src/pages/ClubDetailPage/ClubDetailPage.tsx, frontend/src/pages/ClubDetailPage/LegacyClubDetailPage.tsx |
ClubDetailTopBar import 위치 변경, ClubDetailFooter에 hideShareButtonOnMobile prop 전달, 기타 formatting 및 whitespace 조정 |
Club 상세 페이지 자식 컴포넌트 frontend/src/pages/ClubDetailPage/components/ClubApplyButton/ClubApplyButton.tsx, frontend/src/pages/ClubDetailPage/components/ClubDetailFooter/ClubDetailFooter.tsx |
hideShareButtonOnMobile prop 추가 및 컴포넌트 체인을 통해 전달, 공유 버튼 가시성 조건 재정의 (모바일/태블릿에서 선택적으로 숨김) |
공유 기능 frontend/src/pages/ClubDetailPage/components/ShareButton/ShareButton.tsx |
공유 URL 기본 경로를 https://www.moadong.com/club/에서 https://www.moadong.com/clubDetail/로 변경 |
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
- Revert " [release] FE v1.1.19" #1112: ClubDetailPage 및 ShareButton 관련 컴포넌트에 대한 동일한 파일 수정으로 공유 기능과 직접 연관
- [feature] 공유하기 웹뷰 브리지 복구 #1110: ShareButton.tsx의 공유 로직 수정으로 관련 (URL 변경 및 웹뷰 브릿지 처리)
- [release] FE v1.1.20 #1116: ClubApplyButton 및 LegacyClubDetailPage의 장치 기반 버튼 숨김 처리로 직접 연관
Suggested reviewers
- lepitaaar
- oesnuj
- suhyun113
🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Linked Issues check | PR의 변경 사항이 연결된 이슈 MOA-581의 요구사항과 불일치합니다. 이슈는 앱 1.0.1 버전에서만 공유 버튼을 숨기도록 요구하지만, PR은 LegacyClubDetailPage에서만 숨기고 ClubDetailPage에서는 모든 기기에 표시합니다. | 앱 버전 1.0.1을 감지하는 로직을 구현하고, 해당 버전에서만 공유 버튼을 조건부로 숨기도록 수정하세요. | |
| Out of Scope Changes check | ❓ Inconclusive | CLAUDE.md와 .claude 항목을 .gitignore에 추가하는 변경이 MOA-581 이슈의 범위와 무관해 보입니다. | .gitignore 변경의 목적을 명확히 하고, 필요시 별도의 커밋으로 분리하거나 이슈에 연결하세요. |
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | PR 제목이 공유 버튼 표시 활성화를 명확하게 설명하고 있으며, 변경 사항의 주요 목적을 잘 반영하고 있습니다. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
- 📝 Generate docstrings
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.
#️⃣연관된 이슈
📝작업 내용
LegacyClubDetailPage에서만 공유 버튼 숨김 조건을 유지하고,ClubDetailPage에서는 모든 기기에서 공유 버튼이 표시되도록 변경하였습니다.club/id에서clubDetail/id로 변경했습니다.중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
릴리스 노트
새로운 기능
개선 사항
✏️ Tip: You can customize this high-level summary in your review settings.