Skip to content

Docs: GA 페이지 경로 추가#268

Merged
heeeeyong merged 2 commits into
developfrom
chore/minor-updates
Sep 30, 2025
Merged

Docs: GA 페이지 경로 추가#268
heeeeyong merged 2 commits into
developfrom
chore/minor-updates

Conversation

@heeeeyong

@heeeeyong heeeeyong commented Sep 30, 2025

Copy link
Copy Markdown
Collaborator

#️⃣연관된 이슈

없음

📝작업 내용

image

  • GA 페이지 경로가 부족해 좀 더 세분화해서 추가했습니다.

💬리뷰 요구사항

없음

Summary by CodeRabbit

  • 뉴 기능
    • 페이지 제목 규칙을 URL 첫 세그먼트 기반으로 확장하여 신규 경로(회원가입, 글쓰기, 기타 피드, 내 피드, 팔로우, 오늘의 단어 등)를 지원하고, 그룹 상세 페이지 제목을 ‘THIP - 모임 상세 페이지’로 개선했습니다. 매칭되지 않을 경우 기본값은 ‘THIP’가 적용됩니다.
    • 마이페이지의 메뉴 표시 텍스트를 ‘버젼 1.0.0’에서 ‘버젼 1.3.0’으로 업데이트했습니다.

@heeeeyong heeeeyong self-assigned this Sep 30, 2025
@heeeeyong heeeeyong added the 📃 Docs Improvements or additions to documentation label Sep 30, 2025
@vercel

vercel Bot commented Sep 30, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
thip Ready Ready Preview Comment Sep 30, 2025 4:07am

@coderabbitai

coderabbitai Bot commented Sep 30, 2025

Copy link
Copy Markdown

Walkthrough

라우트 첫 세그먼트 기준으로 페이지 타이틀 매핑 로직을 재구성하고 신규 경로(signup, post, otherfeed, myfeed, follow, today-words)를 추가했으며 /group 타이틀을 수정했습니다. 매칭 실패 시 ‘THIP’로 폴백 후 페이지뷰를 전송하는 흐름은 유지됩니다. 마이페이지 메뉴 버튼 라벨을 1.3.0으로 갱신했습니다.

Changes

Cohort / File(s) Summary
라우트 기반 타이틀 로직 업데이트
src/components/common/Layout.tsx
URL 첫 세그먼트 단위로 정규식 매칭 그룹화. titleRules에 signup, post, otherfeed, myfeed, follow, today-words 추가. /group 매핑을 'THIP - 모임 상세 페이지'로 변경. 매칭 실패 시 'THIP'로 설정 후 페이지뷰 전송 로직 유지. 주석 보완.
마이페이지 라벨 변경
src/pages/mypage/Mypage.tsx
MenuButton 표시 텍스트를 "버젼 1.0.0"에서 "버젼 1.3.0"으로 변경. 동작 로직 변화 없음.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant U as User
  participant R as Router
  participant L as Layout
  participant T as TitleResolver
  participant D as Document
  participant A as Analytics

  U->>R: 경로 이동
  R->>L: route 변경 통지
  L->>T: 첫 세그먼트 추출 및 규칙 매칭
  T-->>L: 문서 타이틀 결정 (또는 'THIP' 폴백)
  L->>D: document.title 설정
  L->>A: sendPageView 호출
  A-->>L: 확인
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • ho0010
  • ljh130334

Poem

버니는 길을 묻고, 첫 칸을 본다
타이틀은 척— 정하고, 뷰는 휙— 보낸다
모임엔 새 이름, 사인업도 인사
마이페이지 숫자도 살짝 새 단장
폴짝! 로그의 들판에 발자국 두 개 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed 이 제목은 GA 페이지 경로를 세분화하여 추가하는 주요 변경 사항을 간결하고 명확하게 요약하고 있어 변경의 핵심 의도를 한눈에 파악할 수 있습니다.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/minor-updates

📜 Recent 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 1373000 and ecac20c.

📒 Files selected for processing (2)
  • src/components/common/Layout.tsx (1 hunks)
  • src/pages/mypage/Mypage.tsx (1 hunks)
🔇 Additional comments (2)
src/pages/mypage/Mypage.tsx (1)

146-146: 버전 라벨 갱신 확인 완료

마이페이지 메뉴에서 노출되는 버전 라벨이 최신값(1.3.0)으로 반영된 것 확인했습니다. 사용자 혼선이 줄어들 것으로 보입니다.

src/components/common/Layout.tsx (1)

10-26: 새 경로 타이틀 매핑 반영 잘 확인했습니다

첫 세그먼트 기반으로 경로를 세분화하면서 GA 타이틀 매칭이 깔끔해졌고, 신규 경로들도 누락 없이 커버되는 것으로 보입니다. Fallback 흐름도 이전과 동일해서 안정적으로 동작할 듯합니다.


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

@heeeeyong heeeeyong merged commit 7b9e1c8 into develop Sep 30, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📃 Docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant