[feature] 앱 다운로드 배너 트래킹에 A/B 테스트 그룹 정보 추가#1056
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| Cohort / File(s) | 변경 요약 |
|---|---|
배너 AB 테스트 추적 강화 frontend/src/pages/MainPage/components/Banner/Banner.tsx |
getABTestGroup 함수 import 추가 및 APP_STORE_LINK 클릭 추적 이벤트에 abTestGroup 값 포함 |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related issues
- [feature] MOA-530 배너에 AB 속성을 추가한다 #1055: 배너에 AB 테스트 그룹 데이터를 추가하는 동일한 목표의 변경사항으로, 배너 추적 로직에 AB 그룹 값을 포함하는 점에서 연관됨
Possibly related PRs
- [release] 앱 출시 팝업 릴리즈 #1011: 메인 PR에서 사용되는
getABTestGroup함수를 내보내는 AB 테스트 로직 구현에 직접 의존 - [feature] 앱 출시 배너를 추가한다 #938: 배너 앱스토어 링크 클릭 추적 로직을 수정하며, 메인 PR은 해당 추적 페이로드에
abTestGroup을 추가 - [feature] 메인화면에 앱 출시 팝업을 추가한다 #1010: 메인 PR이 Popup 모듈에서
getABTestGroup을 import하여 사용하며, 해당 함수의 내보내기 변경과 직접 관련
Suggested reviewers
- lepitaaar
- oesnuj
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | PR 제목이 변경사항의 주요 목적을 명확하게 설명하고 있습니다. 앱 다운로드 배너 트래킹에 A/B 테스트 그룹 정보를 추가한다는 내용이 코드 변경사항(getABTestGroup 추가, abTestGroup을 추적 페이로드에 포함)과 정확히 일치합니다. |
| Linked Issues check | ✅ Passed | 연결된 이슈 MOA-530은 배너에 A/B 속성을 추가하는 것을 목표로 하며, PR의 변경사항(Banner.tsx에서 getABTestGroup 호출 후 abTestGroup을 추적 이벤트에 포함)이 이 요구사항을 충족합니다. |
| Out of Scope Changes check | ✅ Passed | 모든 변경사항이 MOA-530의 범위 내입니다. Banner.tsx에 추가된 변경은 A/B 테스트 그룹 정보를 APP_STORE_LINK 처리에 포함하는 것으로, 제시된 목표와 완전히 일치합니다. |
| 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.
suhyun113
left a comment
There was a problem hiding this comment.
A/B 테스트에 필요한 정보를 잘 추가해주셨네요!
덕분에 팝업과 배너 각각의 전환 효과를 분리해서 분석할 수 있고, 캐니벌라이제이션 여부도 명확히 확인 가능할 것 같습니다! 수고하셨서여~
#️⃣연관된 이슈
📝작업 내용
메인 페이지 앱 다운로드 배너 클릭 이벤트에 abTestGroup property를 추가하여 팝업 A/B 테스트의 정확한 효과 측정이 가능하도록 개선했습니다.
배경
현재 메인 페이지에서 앱 다운로드를 유도하는 두 가지 경로가 있습니다
팝업은 A/B 테스트 중이며, 사용자의 50%에게만 노출되고 있습니다:
Problem
현재 배너 클릭 이벤트(
APP_DOWNLOAD_BANNER_CLICKED)에는 A/B 테스트 그룹 정보가 없어, 다음과 같은 핵심 질문에 답할 수 없었습니다.중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.