[feature] MOA-1076 동아리 상세 기본 탭을 활동사진으로 변경 - #1969
Conversation
체류시간 중앙값이 7초(3초 이하 30%, 10초 이하 60%)라 첫 화면이 판단을 좌우한다. 유저가 능동적으로 이동하는 탭도 활동사진이 3,451건으로 행사일정 1,471건의 2.3배다. 활동사진이 없는 동아리는 기존대로 소개내용으로 폴백하고, URL의 tab 파라미터 우선순위는 그대로 둔다.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ UI 변경사항 없음
전체 172개 스토리 · 64개 컴포넌트 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. Walkthrough동아리 상세 페이지는 유효한 Changes동아리 상세 탭
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The detail page now opens on activity photos when available and falls back to the introduction tab otherwise, while explicit URL tab selections continue to work; no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
#️⃣연관된 이슈
#1968
📝작업 내용
상세페이지 진입 시 기본 탭을
소개내용→활동사진으로 바꾼다. 6줄 변경이다.소개내용으로 폴백?tab=파라미터 우선순위는 그대로 유지if (!clubDetail) return null로 아무것도 렌더되지 않아 탭이 깜빡이지 않는다🫡 참고사항 — 근거 (30일 실측)
동아리를 고르는 페이지에서 중앙값 7초는 읽지 않았다는 뜻이다. 그 7초에 텍스트 소개가 떠 있고, 유저가 스스로 찾아가는 곳은 사진이다.
마지막 줄(37% vs 15.8%)은 상관이지 인과가 아니다. 이미 관심 있는 사람이 사진도 보고 지원도 하는 것일 수 있다. 근거로 쓰되 효과 크기를 이 수치로 기대하면 안 된다.
논의하고 싶은 부분(선택)
A/B 없이 적용하는 판단. 상세 방문 유저가 월 1,509명(일 50명)이라, 기준선 15.8%에서 20% 상대 리프트를 검출하려면 53일이 걸린다. 한 줄 변경에 두 달을 묶는 건 비용이 안 맞고 되돌리는 비용도 거의 없어서 적용 후 관찰로 정했다. 3월 신학기(주 2,462명, 5.6배)에는 10일이면 되므로, 근거가 더 약한 안들은 그때 실험으로 돌린다.
테스트를 안 붙였다. 이 저장소에 전체 페이지 단위 테스트 선례가 없고(기존 7개는 전부 컴포넌트 단위), 이 변경을 테스트하려면 router·react-query·mixpanel·naver map을 다 목킹해야 해서 6줄 변경에 비해 과하다고 판단했다. 분기 자체는
hasFeeds ? PHOTOS : INTRO와tabParam우선순위뿐이고 타입체크로 덮인다. 붙이는 게 낫다고 보면 알려달라.측정 시 주의
변경 후에는
Club Feed Tab Clicked가 줄고Club Intro Tab Clicked가 는다. 기본 탭이 바뀌어 클릭의 의미 자체가 달라지기 때문이다. 효과 판정은 탭 클릭이 아니라Club Apply Button Clicked/ClubDetailPage Visited로 봐야 한다.상위 스토리는 MOA-1075다.
검증:
npm run typecheck통과 · ESLint 0 errors ·npx jest460개 / 55개 스위트 통과Summary by CodeRabbit