Skip to content

[SETTING/#58] firebase Analytics - #61

Merged
oilbeaneda merged 3 commits into
developfrom
setting/#58-firebase
Jul 22, 2026
Merged

oilbeaneda merged 3 commits into
developfrom
setting/#58-firebase

Conversation

@oilbeaneda

@oilbeaneda oilbeaneda commented Jul 22, 2026 •

Copy link
Copy Markdown
Member

Related issue 🛠

Work Description ✏️

Firebase Analytics SDK 를 붙여서 설치/DAU/리텐션은 자동으로 잡히게 합니다.

Screenshot 📸

image

Uncompleted Tasks 😅

  • Task1

Summary by CodeRabbit

  • 새 기능

    • Firebase Analytics를 연동해 주요 사용자 행동을 측정할 수 있습니다.
    • 검색, 필터 적용, 지도 링크 클릭, 상세 조회, 즐겨찾기, 제보 제출 및 온보딩 완료 이벤트가 기록됩니다.
    • 분석 데이터에 검색 결과 수, 선택 항목 수 등 관련 정보가 함께 수집됩니다.
  • 개선

    • 앱 빌드 버전 코드가 2에서 3으로 업데이트되었습니다.

@oilbeaneda oilbeaneda self-assigned this Jul 22, 2026
@oilbeaneda oilbeaneda added the ⚙️ Setting feat - 기본 세팅 label Jul 22, 2026
@coderabbitai

coderabbitai Bot commented Jul 22, 2026 •

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Firebase Analytics 의존성과 공통 로깅 계층을 추가하고, 홈·검색·온보딩·제보 흐름에서 주요 이벤트와 파라미터를 기록하도록 변경했습니다. 앱 versionCode를 3으로 올렸습니다.

Changes

Firebase Analytics

Layer / File(s) Summary
Analytics 계약 및 Firebase 연결
app/build.gradle.kts, app/src/main/java/org/app/core/analytics/*, app/src/main/java/org/app/core/analytics/di/*, gradle/libs.versions.toml
AnalyticsHelper와 이벤트·파라미터 상수를 정의하고, Firebase 구현체 및 Hilt 바인딩을 추가했습니다. Firebase 의존성과 Google Services 플러그인을 활성화했으며 versionCode를 3으로 변경했습니다.
홈 및 검색 이벤트 로깅
app/src/main/java/org/app/presentation/home/HomeViewModel.kt, app/src/main/java/org/app/presentation/home/homesearch/HomeSearchViewModel.kt
필터 적용, 지도 링크 클릭, 펍 상세 조회, 즐겨찾기 추가, 검색 결과 성공 시 Analytics 이벤트를 기록합니다.
온보딩 및 제보 이벤트 로깅
app/src/main/java/org/app/presentation/onboarding/signup/SignUpViewModel.kt, app/src/main/java/org/app/presentation/mypage/report/ReportViewModel.kt
온보딩 완료와 제보 제출 성공 시 이벤트 및 팀 수·카테고리 파라미터를 기록합니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant ViewModel
  participant AnalyticsHelper
  participant FirebaseAnalyticsHelper
  participant FirebaseAnalytics
  ViewModel->>AnalyticsHelper: logEvent(name, params)
  AnalyticsHelper->>FirebaseAnalyticsHelper: 구현체 호출
  FirebaseAnalyticsHelper->>FirebaseAnalytics: 이벤트 및 파라미터 전송
Loading

Suggested labels: Feat ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Firebase Analytics SDK 통합으로 자동 수집 요구를 충족하고 관련 로깅/DI 추가도 이 목적과 일치합니다.
Out of Scope Changes check ✅ Passed 추가된 커스텀 이벤트 로깅과 DI, 버전코드 증가는 Firebase Analytics 통합 범위 안의 변경으로 보입니다.
Title check ✅ Passed 제목이 Firebase Analytics 추가라는 핵심 변경을 직접적으로 나타내며, 변경 내용과 관련성이 높습니다.
Description check ✅ Passed 템플릿의 관련 이슈, 작업 내용, 스크린샷, 미완료 작업 섹션이 모두 포함되어 있어 대부분 충족합니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch setting/#58-firebase

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.

@oilbeaneda
oilbeaneda merged commit 7edff60 into develop Jul 22, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️ Setting feat - 기본 세팅

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Setting] Firebase Analytics

1 participant