[Y26W2-427] chore(web): prior high event parameter setting #202

gonn-i · 2025-11-26T11:47:15Z

✨ 변경 사항

✅ 체크리스트

기능 동작 확인
코드 리뷰 반영
테스트 통과
UI/UX 확인

📸 스크린샷 (선택)

📝 기타 참고 사항

Summary by CodeRabbit

릴리스 노트

Chores
- 분석 데이터 수집 구성을 업데이트했습니다.
- 사용자 인증 및 보드 작업 관련 이벤트 추적 시스템을 개선했습니다.
- 초대 및 공유 기능 관련 분석 추적을 강화했습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

github-actions · 2025-11-26T11:47:24Z

🎫 관련 Jira 티켓 🔗: Y26W2-427

📝 Jira 티켓 정보:

제목: [이벤트] 로그 작업 (2/2)
상태: 검토 중
담당자: 김고은

coderabbitai · 2025-11-26T11:47:26Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📋 Walkthrough

분석 이벤트 추적 시스템을 리팩토링하여 CLICK_FILTER 이벤트를 제거하고 SIGNUP, LOGIN, LOGOUT, SIGNOUT, BOARD_CREATE, BOARD_URL_INVITE 새로운 이벤트 매핑을 추가하였습니다. 각 이벤트에 대응하는 파라미터 팩토리 함수들을 추가하였으며, FilterName 타입을 LoginMethod 타입으로 교체하였습니다.

📊 Changes

Cohort / File(s)	변경 사항
Event 매핑 및 타입 정의 `apps/web/src/shared/contants/analytics/mixpanel-event-parameter-map.ts`	CLICK_FILTER 이벤트 제거 및 SIGNUP, LOGIN, LOGOUT, SIGNOUT, BOARD_CREATE, BOARD_URL_INVITE 이벤트 매핑 추가
타입 정의 `apps/web/src/shared/contants/analytics/property-type.ts`	FilterName 타입 제거, LoginMethod 타입 추가
인증 관련 파라미터 함수 `apps/web/src/shared/contants/analytics/parameters/make-signup-parameter.ts`, `make-login-parameter.ts`, `make-logout-parameter.ts`, `make-signout-parameter.ts`	회원가입, 로그인, 로그아웃, 로그아웃 이벤트 파라미터 팩토리 함수 신규 추가
보드/테이블 관련 파라미터 함수 `apps/web/src/shared/contants/analytics/parameters/make-board-create-parameter.ts`, `make-board-url-invite-parameter.ts`, `make-table-url-share.ts`, `make-table_create.ts`	보드 생성, 보드 URL 초대, 테이블 URL 공유, 테이블 생성 이벤트 파라미터 함수 신규 추가
추가 파라미터 함수 `apps/web/src/shared/contants/analytics/parameters/make-board-group-view.ts`, `make-hotel-add.ts`	보드 그룹 뷰 및 호텔 추가 이벤트 파라미터 함수 신규 추가
삭제된 파일 `apps/web/src/shared/contants/analytics/parameters/make-click-filter-parameter.ts`	파일 삭제

🎯 Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

새로운 파라미터 팩토리 함수들이 여러 파일에 걸쳐 추가되었으며, 각 함수의 반환 객체 구조가 올바른지 확인 필요
mixpanel-event-parameter-map.ts의 이벤트 매핑이 모든 새 함수와 일치하는지 검증 필요
삭제된 make-click-filter-parameter.ts가 다른 곳에서 참조되지 않는지 확인
각 파라미터 함수의 타입 정의가 정확한지 검토 필요

🐰 Poem

🥕 필터는 사라지고, 새로운 이벤트들 등장하네
회원가입부터 보드까지, 추적 기능 쌩~
팩토리 함수들 쭉쭉 늘어나며
분석은 더욱 정교해지는 걸요! ✨
토끼가 춤을 춘다는 것!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	PR 설명이 템플릿 구조는 따르고 있으나, '변경 사항' 섹션에 구체적인 내용이 없고 모든 체크리스트 항목이 미체크 상태로 남아있어 실질적인 정보가 부족합니다.	변경 사항 섹션에 구체적인 내용을 작성하고, 완료된 체크리스트 항목에 체크를 표시한 후 필요시 참고 사항을 추가해주세요.
Title check	❓ Inconclusive	PR 제목이 실제 변경 사항의 주요 내용(분석 이벤트 파라미터 리팩토링)과 부분적으로 관련되어 있으나, 구체적인 변경 내용을 명확하게 설명하지 않아 'prior high event parameter setting'이라는 표현이 모호합니다.	제목을 더 구체적으로 수정하여 mixpanel 이벤트 파라미터 업데이트 내용을 명확히 해주세요. 예: 'Y26W2-427 chore(web): update mixpanel analytics event parameters'

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

Provide your own instructions using the high_level_summary_instructions setting.
Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

📝 Description — Summarize the main change in 50–60 words, explaining what was done.

📓 References — List relevant issues, discussions, documentation, or related PRs.

📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.

📊 Contributor Summary — Include a Markdown table showing contributions:
| Contributor | Lines Added | Lines Removed | Files Changed |

✔️ Additional Notes — Add any extra reviewer context.
Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.

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.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (1)

apps/web/src/shared/contants/analytics/parameters/make-table_create.ts (1)

1-11: 파일 네이밍을 다른 파라미터 유틸과 통일하는 것을 고려해주세요.

내용 자체는 문제 없지만, 파일명이 make-table_create.ts로 다른 유틸(make-board-create-parameter.ts, make-signup-parameter.ts 등)과 네이밍 규칙이 살짝 달라 보입니다. make-table-create-parameter.ts처럼 맞춰두면 검색·유지보수에 더 수월할 것 같습니다.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3648427 and 3484678.

📒 Files selected for processing (13)

apps/web/src/shared/contants/analytics/mixpanel-event-parameter-map.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-board-create-parameter.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-board-group-view.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-board-url-invite-parameter.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-click-filter-parameter.ts (0 hunks)
apps/web/src/shared/contants/analytics/parameters/make-hotel-add.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-login-parameter.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-logout-parameter.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-signout-parameter.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-signup-parameter.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-table-url-share.ts (1 hunks)
apps/web/src/shared/contants/analytics/parameters/make-table_create.ts (1 hunks)
apps/web/src/shared/contants/analytics/property-type.ts (1 hunks)

💤 Files with no reviewable changes (1)

apps/web/src/shared/contants/analytics/parameters/make-click-filter-parameter.ts

🧰 Additional context used

🧬 Code graph analysis (2)

apps/web/src/shared/contants/analytics/mixpanel-event-parameter-map.ts (6)

apps/web/src/shared/contants/analytics/parameters/make-signup-parameter.ts (1)

makeSignupParameter (1-3)

apps/web/src/shared/contants/analytics/parameters/make-login-parameter.ts (1)

makeLoginParameter (3-5)

apps/web/src/shared/contants/analytics/parameters/make-logout-parameter.ts (1)

makeLogoutParameter (1-3)

apps/web/src/shared/contants/analytics/parameters/make-signout-parameter.ts (1)

makeSignoutParameter (1-3)

apps/web/src/shared/contants/analytics/parameters/make-board-create-parameter.ts (1)

makeBoardCreateParameter (1-3)

apps/web/src/shared/contants/analytics/parameters/make-board-url-invite-parameter.ts (1)

makeBoardUrlInviteParameter (1-3)

apps/web/src/shared/contants/analytics/parameters/make-login-parameter.ts (1)

apps/web/src/shared/contants/analytics/property-type.ts (1)

LoginMethod (1-1)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

GitHub Check: build
GitHub Check: deploy

🔇 Additional comments (11)

apps/web/src/shared/contants/analytics/parameters/make-hotel-add.ts (1)
1-10: _hotel_count 파라미터가 반환 객체에서 제외되는 이유를 확인해주세요.

함수 시그니처에서 _hotel_count 파라미터를 받지만, 반환 객체에는 포함하지 않고 있습니다. 언더스코어 접두사는 의도적으로 사용하지 않는다는 것을 나타내지만, 이것이 의도된 동작인지 확인이 필요합니다.

만약 hotel_count가 실제로 필요하다면 다음과 같이 수정을 고려해주세요:
 export const makeHotelAddParameter = (
   board_id: number,
   hotel_id: number,
   hotel_domain: string,
-  _hotel_count: number,
+  hotel_count: number,
 ) => ({
   board_id,
   hotel_id,
   hotel_domain,
+  hotel_count,
 });
반대로, 이 파라미터가 불필요하다면 함수 시그니처에서 완전히 제거하는 것이 좋습니다.
apps/web/src/shared/contants/analytics/parameters/make-table-url-share.ts (1)

1-3: LGTM!

간결하고 명확한 파라미터 팩토리 함수입니다. 구현이 올바르고 패턴도 일관성 있게 작성되었습니다.

apps/web/src/shared/contants/analytics/parameters/make-board-group-view.ts (1)

1-9: LGTM!

파라미터 팩토리 함수가 올바르게 구현되었습니다. 다른 파라미터 빌더들과 일관된 패턴을 따르고 있습니다.

apps/web/src/shared/contants/analytics/parameters/make-signup-parameter.ts (1)

1-3: LGTM!

회원가입 이벤트 파라미터 팩토리가 올바르게 구현되었습니다.

apps/web/src/shared/contants/analytics/parameters/make-signout-parameter.ts (1)

1-3: LGTM!

회원 탈퇴 이벤트 파라미터 팩토리가 올바르게 구현되었습니다.

apps/web/src/shared/contants/analytics/parameters/make-logout-parameter.ts (1)

1-3: LGTM!

로그아웃 이벤트 파라미터 팩토리가 올바르게 구현되었습니다.

apps/web/src/shared/contants/analytics/parameters/make-board-create-parameter.ts (1)

1-3: LGTM!

보드 생성 이벤트 파라미터 팩토리가 올바르게 구현되었습니다. 일관된 패턴을 따르고 있습니다.

apps/web/src/shared/contants/analytics/property-type.ts (1)

1-1: 코드는 현재 요구사항에 맞게 올바르게 구현되었습니다.

검증 결과, LoginMethod는 현재 카카오만 지원하고 있으며 이는 백엔드 API(@ssok/api)에서도 카카오 OAuth 엔드포인트만 구현되어 있기 때문에 올바른 구현입니다. 또한 이 타입은 인증 로직이 아닌 분석 이벤트 추적을 위한 타입이므로, 향후 다른 로그인 방법이 백엔드에 추가될 때 함께 확장하면 됩니다. 현재 코드는 제한적이지 않으며, 구현된 기능에 정확하게 매칭되어 있습니다.

apps/web/src/shared/contants/analytics/parameters/make-login-parameter.ts (1)

1-5: 로그인 파라미터 빌더 구현 적절합니다.

LoginMethod 타입을 그대로 래핑해서 { login_method } 객체를 만드는 단순한 유틸로, 다른 파라미터 빌더들과 패턴도 잘 맞습니다.

apps/web/src/shared/contants/analytics/parameters/make-board-url-invite-parameter.ts (1)

1-3: 보드 URL 초대 파라미터 유틸이 간결하고 명확합니다.

board_id만 감싼 순수 함수라 재사용성과 타입 안정성이 좋고, 다른 analytics 파라미터 유틸들과 일관적입니다.

apps/web/src/shared/contants/analytics/mixpanel-event-parameter-map.ts (1)

1-24: 이벤트 파라미터/이름 매핑이 잘 정리되었습니다.

각 이벤트별로 전용 파라미터 빌더를 import 해서 EventParameterMap에 매핑하고, 동일 키를 EventNameMapper에도 사용해 일관성이 좋습니다. 현재 키 집합도 두 객체 간에 잘 맞춰져 있어서 타입 추론과 사용성이 괜찮아 보입니다.

github-actions · 2025-11-26T11:51:27Z

This pull request (commit 3484678) has been deployed to Vercel ▲ - View GitHub Actions Workflow Logs

Name	Link
🌐 Unique	https://ssok-c7r9jq24q-sudosubin-team.vercel.app
🔍 Inspect	https://vercel.com/sudosubin-team/ssok/7eN8fgRqVKdrxSuDtpKoVx1ssdoz

github-actions · 2025-11-27T16:44:29Z

This pull request (commit 186c8fe) has been deployed to Vercel ▲ - View GitHub Actions Workflow Logs

Name	Link
🌐 Unique	https://ssok-ikbst64oa-sudosubin-team.vercel.app
🔍 Inspect	https://vercel.com/sudosubin-team/ssok/HiMnLnheYP3PcPMCGiwQCETLuVjv

[Y26W2-427] chore(web): prior high event parameter setting

3484678

github-actions bot assigned gonn-i Nov 26, 2025

github-actions bot requested a review from sudosubin November 26, 2025 11:47

github-actions bot temporarily deployed to Preview November 26, 2025 11:47 Inactive

github-actions bot added size/s apps/web labels Nov 26, 2025

gonn-i marked this pull request as draft November 26, 2025 11:47

github-actions bot added size/s and removed size/s labels Nov 26, 2025

coderabbitai bot reviewed Nov 26, 2025

View reviewed changes

gonn-i added 6 commits November 27, 2025 19:16

[Y26W2-426] chore(web): 이벤트 속성 수정사항 반영

7e4800d

[Y26W2-426] feat: 로그아웃 회원탈퇴시 이벤트 수집

e8db08e

[Y26W2-426] feat: 여행 보드 생성시 eventTrack

527aab3

[Y26W2-426] feat(webview): 멤버 초대하기 eventTrack

d91fccd

[Y26W2-426] feat(web): 비교표 링크 복사 event track

c68581f

[Y26W2-426] feat(web): 숙소 추가 및 비교표 생성 event track

186c8fe

github-actions bot temporarily deployed to Preview November 27, 2025 16:41 Inactive

github-actions bot added size/m and removed size/s labels Nov 27, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Y26W2-427] chore(web): prior high event parameter setting #202

[Y26W2-427] chore(web): prior high event parameter setting #202

Uh oh!

gonn-i commented Nov 26, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

github-actions bot commented Nov 26, 2025 •

edited by atlassian bot

Loading

Uh oh!

coderabbitai bot commented Nov 26, 2025 •

edited by atlassian bot

Loading

Review skipped

Uh oh!

coderabbitai bot left a comment

Uh oh!

github-actions bot commented Nov 26, 2025

Uh oh!

github-actions bot commented Nov 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

[Y26W2-427] chore(web): prior high event parameter setting #202

Are you sure you want to change the base?

[Y26W2-427] chore(web): prior high event parameter setting #202

Uh oh!

Conversation

gonn-i commented Nov 26, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✨ 변경 사항

✅ 체크리스트

📸 스크린샷 (선택)

📝 기타 참고 사항

Summary by CodeRabbit

릴리스 노트

Uh oh!

github-actions bot commented Nov 26, 2025 • edited by atlassian bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai bot commented Nov 26, 2025 • edited by atlassian bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

📋 Walkthrough

📊 Changes

🎯 Estimated code review effort

🐰 Poem

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Nov 26, 2025

Uh oh!

github-actions bot commented Nov 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

gonn-i commented Nov 26, 2025 •

edited by coderabbitai bot

Loading

github-actions bot commented Nov 26, 2025 •

edited by atlassian bot

Loading

coderabbitai bot commented Nov 26, 2025 •

edited by atlassian bot

Loading