Skip to content

Comments

[feature] Mixpanel IP 추적하도록 변경 및 ignore_dnt 적용#850

Merged
seongwon030 merged 1 commit intodevelop-fefrom
feature/#849-mixpanel-ip-tracking-MOA-348
Nov 19, 2025
Merged

[feature] Mixpanel IP 추적하도록 변경 및 ignore_dnt 적용#850
seongwon030 merged 1 commit intodevelop-fefrom
feature/#849-mixpanel-ip-tracking-MOA-348

Conversation

@seongwon030
Copy link
Member

@seongwon030 seongwon030 commented Nov 17, 2025

#️⃣연관된 이슈

ex) #849

📝작업 내용

  • mixpanel.init 옵션에 ignore_dnt: true 추가하여 브라우저 DNT(Do not Track) 설정을 무시하고 추적하도록 변경했습니다.
  • ip: false설정을 제거하여 ip 추적 방지를 해제했습니다.

이전로그

스크린샷 2025-11-18 오후 4 37 03

변경후 로그

스크린샷 2025-11-18 오후 4 37 30

Active Users

스크린샷 2025-11-18 오후 4 37 55

중점적으로 리뷰받고 싶은 부분(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

릴리스 노트

  • 기타
    • 분석 추적 설정이 업데이트되었습니다. Do Not Track이 활성화된 경우에도 분석 데이터 수집이 계속됩니다.

- ignore_dnt: true 설정 추가
- debug: false 설정 추가
@seongwon030 seongwon030 self-assigned this Nov 17, 2025
@seongwon030 seongwon030 added ✨ Feature 기능 개발 💻 FE Frontend labels Nov 17, 2025
@vercel
Copy link

vercel bot commented Nov 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
moadong Ready Ready Preview Comment Nov 17, 2025 6:14pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 17, 2025

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

개요

frontend/src/utils/initSDK.ts 파일의 Mixpanel 초기화 함수에서 설정 옵션을 변경했습니다. ip: false에서 ignore_dnt: true로 변경되어 Do Not Track 상태에서도 트래킹이 진행되도록 수정되었습니다.

변경사항

변경 유형 파일 설명
설정 변경 frontend/src/utils/initSDK.ts Mixpanel 초기화 설정에서 ip: false에서 ignore_dnt: true로 변경하여 Do Not Track 활성화 상태에서도 트래킹 진행

추정 코드 리뷰 시간

🎯 1 (Trivial) | ⏱️ ~3분

  • 변경 범위가 매우 제한적 (단일 파일)
  • 제어 흐름 변경 없음
  • 설정 값 변경만 해당
  • 주의: 연결된 이슈(MOA-348)에서 계획한 ip: false → true 변경과 실제 변경(ignore_dnt: true)이 다른지 검토 필요

관련 가능성 있는 이슈

제안하는 라벨

💻 FE

제안하는 리뷰어

  • lepitaaar
  • oesnuj
  • suhyun113

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check ❓ Inconclusive PR 제목은 '[feature] Mixpanel IP 추적하도록 변경 및 ignore_dnt 적용'이며, 변경사항과 부분적으로 관련되어 있습니다. 제목에서 'IP 추적하도록 변경'은 실제 변경사항의 일부를 반영하지만, 실제 코드 변경의 핵심은 'ip: false → ignore_dnt: true'이고, 이는 IP 추적 활성화보다는 Do Not Track 무시 설정에 관한 것입니다. 제목을 더 명확하게 수정하여 실제 변경사항을 정확히 반영하도록 제안합니다. 예: '[feature] Mixpanel 설정: IP 추적 활성화 및 DNT 무시'로 변경하거나, 변경사항과 의도를 더 정확히 일치시키기 바랍니다.
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed PR의 코드 변경(ip: false → ignore_dnt: true)이 연관 이슈 MOA-348의 요구사항(ip:false → true로 변경)과 일치합니다.
Out of Scope Changes check ✅ Passed PR의 모든 변경은 믹스패널 IP 추적 설정 변경이라는 정의된 범위 내에 있으며, 범위를 벗어난 변경은 없습니다.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/#849-mixpanel-ip-tracking-MOA-348

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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 1e5be34 and 6e7b5e2.

📒 Files selected for processing (1)
  • frontend/src/utils/initSDK.ts (1 hunks)

@seongwon030 seongwon030 changed the title [feature] 믹스패널 ip 추적 방지를 해제한다 [feature] Mixpanel IP 추적 방지 해제 및 ignore_dnt 적용 Nov 18, 2025
@seongwon030 seongwon030 requested a review from oesnuj November 18, 2025 07:36
@seongwon030 seongwon030 linked an issue Nov 18, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@lepitaaar lepitaaar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ip 수집 안하는거 좋아요~
ip false와 ignore_dnt 필드 뭐가다르고 왜 수집되고 있었을까요?

@seongwon030
Copy link
Member Author

seongwon030 commented Nov 19, 2025

ip 수집 안하는거 좋아요~ ip false와 ignore_dnt 필드 뭐가다르고 왜 수집되고 있었을까요?

수집 안 하는걸 해제했어요.. dnt는 특정 브라우저에서 do not track 속성이 있어도 트래킹하겠다는 뜻입니다

이슈: mixpanel/mixpanel-js#179
릴리즈: https://github.com/mixpanel/mixpanel-js/releases/tag/v2.34.0

@seongwon030 seongwon030 changed the title [feature] Mixpanel IP 추적 방지 해제 및 ignore_dnt 적용 [feature] Mixpanel IP 추적하도록 변경 및 ignore_dnt 적용 Nov 19, 2025
@seongwon030 seongwon030 merged commit 07163ee into develop-fe Nov 19, 2025
3 of 5 checks passed
@seongwon030 seongwon030 deleted the feature/#849-mixpanel-ip-tracking-MOA-348 branch December 17, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FE Frontend ✨ Feature 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature] MOA-348 Mixpanel IP 추적 방지 해제 및 ignore_dnt 적용

2 participants