Skip to content

develp branch 작업내용 머지 : develop -> main#261

Merged
heeeeyong merged 9 commits into
mainfrom
develop
Sep 26, 2025
Merged

develp branch 작업내용 머지 : develop -> main#261
heeeeyong merged 9 commits into
mainfrom
develop

Conversation

@heeeeyong
Copy link
Copy Markdown
Collaborator

@heeeeyong heeeeyong commented Sep 26, 2025

이하동일 (ver1.3.0)

Summary by CodeRabbit

  • 신규 기능
    • 알림 클릭 시 서버 확인을 거쳐 관련 화면(피드, 룸, 게시글 등)으로 자동 이동.
    • 메모리 화면 진입 시 전달된 상태에 따라 초기 페이지 필터 적용 및 댓글 바텀시트 자동 열림.
  • 스타일
    • 알림 목록의 스크롤바를 숨겨 시각적 완성도 개선.
    • 사이트 설명 메타태그 추가로 검색·미리보기 노출 개선.
  • 작업
    • GA4 라이브러리 도입 및 이벤트 트래킹 체계 정비.

@heeeeyong heeeeyong self-assigned this Sep 26, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 26, 2025

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

Project Deployment Preview Comments Updated (UTC)
thip Ready Ready Preview Comment Sep 26, 2025 5:02am

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Sep 26, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

메타 설명 태그 추가, GA 통합을 ReactGA 기반으로 교체 및 이벤트 트래킹 추가, 초기 페이지뷰 호출 제거. 알림 확인용 API 래퍼 신설 및 알림 클릭 시 서버 응답에 따른 라우팅 처리. 메모리 페이지에서 라우팅 상태값을 읽어 초기 필터/댓글 바텀시트 오픈 로직 추가. 의존성 react-ga4 추가.

Changes

Cohort / File(s) Summary
GA 통합 교체 및 이벤트 트래킹
src/lib/ga.ts, src/main.tsx
gtag 스크립트 방식 제거 후 ReactGA로 초기화/페이지뷰/이벤트 전송 재구성, 로컬호스트/GA_ID 가드 추가, trackEvent 도입. 앱 시작 시 페이지뷰 자동 전송 제거.
알림 확인 API 및 라우팅 처리
src/api/notifications/postNotificationsCheck.ts, src/pages/notice/Notice.tsx
/notifications/check 호출용 API 래퍼 추가. 알림 카드 클릭 시 응답의 route/params를 해석하여 대상 화면으로 네비게이션.
메모리 페이지 네비게이션 상태 처리
src/pages/memory/Memory.tsx
location.state를 읽어 초기 페이지 필터링 및 댓글 바텀시트 자동 오픈, 이후 state 초기화. 관련 스토어 사용.
메타/설정 업데이트
index.html, package.json
index.html에 한국어 설명 메타 태그 추가. react-ga4@^2.1.0 의존성 추가.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User as 사용자
  participant UI as Notice UI
  participant API as postNotificationsCheck
  participant BE as Backend
  participant Router as Router

  User->>UI: 알림 카드 클릭
  UI->>API: postNotificationsCheck(notificationId)
  API->>BE: POST /notifications/check { notificationId }
  BE-->>API: { isSuccess, data: { route, params } }
  API-->>UI: 응답 반환
  alt isSuccess
    UI->>Router: route/params에 따른 navigate
  else 실패
    UI->>UI: 콘솔 로깅
  end
Loading
sequenceDiagram
  autonumber
  participant App as App Startup
  participant GA as GA(lib)
  participant ReactGA as ReactGA

  App->>GA: initGA()
  alt GA_ID 없거나 localhost
    GA-->>App: 초기화 스킵
  else GA_ID 유효
    GA->>ReactGA: initialize({ anonymizeIp: true })
    ReactGA-->>GA: initialized
  end

  note over App,GA: 페이지 최초 로드 시 자동 pageview 전송 제거
  App->>GA: trackEvent(...) (필요 시)
  GA->>ReactGA: event(...)
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested labels

✨ Feature, 🎨 Html&css

Suggested reviewers

  • ho0010

Poem

툭, 알림을 톡—길이 열렸지
페이지 뷰는 잠시 쉬고, 토끼는 이벤트를 딱!
반짝 GA, 살금살금 추적하니
메모리 숲 댓글꽃 활짝 피네 🌸
메타 한 줄로 향기 더하고
깡총—새 길로, 정확히 뛰어든다 🐇

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

📜 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 9f07391 and 3cb24d0.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (7)
  • index.html (1 hunks)
  • package.json (1 hunks)
  • src/api/notifications/postNotificationsCheck.ts (1 hunks)
  • src/lib/ga.ts (2 hunks)
  • src/main.tsx (1 hunks)
  • src/pages/memory/Memory.tsx (4 hunks)
  • src/pages/notice/Notice.tsx (4 hunks)

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.

@heeeeyong heeeeyong added the 🌏 Deploy 배포 관련 label Sep 26, 2025
@heeeeyong heeeeyong merged commit 39fdfa4 into main Sep 26, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌏 Deploy 배포 관련

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants