템플릿 선택/추천 A/B 테스트 및 GA 이벤트 분리#856
Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 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 |
github-actions
Bot
changed the title
github-actions
Bot
changed the title
JaeIn1
left a comment
JaeIn1
left a comment
There was a problem hiding this comment.
아하 branchLayoutAtom로 분기처리 되고있구나!
현우형 머지하시면 해당 Atom으로 저도 적용시켜 보겠습니다 고생하셨습니다 👍👍
github-actions
Bot
changed the title
| })); | ||
|
|
||
| trackEvent(GA_EVENTS.RETROSPECT.ADD_COMPLETE); | ||
| branchLayout === "A" ? trackEvent(GA_EVENTS.RETROSPECT.ADD_COMPLETE_A_LAYOUT) : trackEvent(GA_EVENTS.RETROSPECT.ADD_COMPLETE_B_LAYOUT); |
There was a problem hiding this comment.
이걸로 A안을 사용하셨는지, B안을 사용하셨는지 이벤트를 추적하는군요!
github-actions
Bot
changed the title
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
브릿지 navigate 사용 시 React Router state가 전달되지 않아 회고 작성 페이지 진입이 실패할 수 있음. 작성하기만 useNavigate로 통일.
- useDesktopBasicModal에 updateState 추가 및 stale closure 수정 - PhaseContext에 goBackToTemplateSelect, detailFrom, isVisibleProgressBar 등 동적 제어 추가 - DetailRetrospectTemplateBranchFunnel 진입 경로별 뒤로가기 및 버튼 노출 분기 처리 - memberSeq null 이슈 수정 및 memberId 타입 number로 변경 - workbox devOptions 로깅 비활성화 Co-Authored-By: klmhyeonwoo <klmhyeonwoo@github.com>
klmhyeonwoo
force-pushed
the
feature/855-a/b-layout
branch
from
74b1003 to
e9828af
Compare
github-actions
Bot
changed the title
4 participants
🏄🏼♂️ Summary (요약)
branchLayout값을 받아 사용자별 UI 분기를 처리합니다.FunnelModal을 도입했습니다.🫨 Describe your Change (변경사항)
branchLayout값에 따라 템플릿 리스트 및 추천 UI가 다르게 표시되도록 변경했습니다.google_analyticsimport 경로를google-analytics로 일관되게 변경했습니다.vite.config.ts에 legacy 플러그인을 추가했습니다.🧐 Issue number and link (참고)
closes: #855
📚 Reference (참조)