Skip to content

fix: GA 분석용 회원 및 링크 식별자 제공 (#335) - #336

Merged
goder-0 merged 2 commits into
mainfrom
feature/335-ga-identifiers
Oct 19, 2026
Merged

goder-0 merged 2 commits into
mainfrom
feature/335-ga-identifiers

Conversation

@goder-0

@goder-0 goder-0 commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

문제와 변경

서버는 DB ID를, 웹은 화면용 Hashids를 GA에 보내 동일 회원과 링크가 분리됩니다. 회원 프로필과 링크 생성·목록·상세·채팅 응답에 DB ID의 10진 문자열 analyticsId를 추가합니다. 기존 id와 API 경로·소유권 검사는 유지합니다.

검증 및 적용

  • Java 17 전체 테스트 586개, build·Checkstyle·JaCoCo 기준 통과. 로컬 H2와 가짜 외부 연동 설정 사용
  • JSON 응답의 기존 Hashids 보존과 서버 이벤트의 회원·링크 ID 일치를 회귀 검증
  • 백엔드를 먼저 배포한 뒤 웹 GA 이벤트에 서버와 동일한 분석용 회원·링크 ID 사용 linkiving#609 적용. 과거 GA 데이터는 소급 변경되지 않음
  • CHANGELOG와 docs/ga-identifiers.md 계약 문서 포함. 운영 배포·실제 GA 수신 검증은 별도

Closes #335

@github-actions

Copy link
Copy Markdown

📊 코드 커버리지 리포트

Overall Project 91.84% 🍏

There is no coverage information present for the Files changed

This was referenced Oct 19, 2026
@goder-0
goder-0 merged commit e560119 into main Oct 19, 2026
3 checks passed
@goder-0

goder-0 commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

최종 검토 및 머지

최종 head 787bcb8의 diff와 호출 경로를 검토했습니다. 기존 Hashids id와 소유권 검사·API 요청 계약을 유지하며, 회원 프로필 및 생성·목록·상세·채팅 링크 응답에 동일한 DB ID 문자열 analyticsId를 제공합니다. 추가 수정이 필요한 사항은 발견하지 못했습니다.

  • 최종 head CI·배포 스크립트 검사·로컬 번들 빌드 성공
  • 로컬 테스트 586개, 실패·오류·생략 0개. H2·모의 외부 연동 기준이며 실제 운영 GA 확인은 별도
  • 필수 팀 승인이 없는 상태임을 안내한 뒤 사용자가 남은 PR을 검토하고 머지하도록 재승인했습니다. 이번 PR에 한해 관리자 우회로 리베이스 머지했으며 저장소 규칙은 변경하지 않았습니다.

머지 커밋: e56011908553cd8396a594dbd1eb21f4f5a30d82.
릴리즈 및 실제 GA 검증은 #337에서 추적합니다. 백엔드 배포 후 프론트를 적용합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

웹·서버 GA 사용자 및 링크 식별자를 DB 기준으로 통일

1 participant