[#453] Firebase Analytics를 적용한다#498
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Code Review
이번 풀 리퀘스트는 앱 내 분석(Analytics) 이벤트를 추적하기 위한 TrackAnalyticsEventUseCase 및 관련 인프라(Firebase Analytics)를 도입하고, 각 뷰모델에서 화면 진입, 할 일 생성/완료, 웹페이지 추가, 푸시 알림 오픈 등의 이벤트를 로깅하도록 구현하였습니다. 코드 리뷰 결과, 할 일 수정 시에도 생성 이벤트가 발생하는 로직 오류, 비옵셔널 타입에 대한 불필요한 옵셔널 바인딩으로 인한 컴파일 에러 가능성, 그리고 UI 라우팅을 처리하는 핸들러의 메인 스레드 안전성 보장(@mainactor 추가)에 대한 개선 사항이 제안되었습니다.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
|
❌ iOS CI build failed. Compiler error lines: Code excerpts: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 연관된 이슈
🎯 의도
screen_view기준의 화면 분석 체계 정리📝 작업 내용
📌 요약
FirebaseAnalyticsCore의존성 추가Domain -> Data -> Infra흐름의 Analytics 계층 구성screen_view,todo_create,todo_complete,webpage_create,push_open추적 연결🔍 상세
TrackAnalyticsEventUseCase기반의 Analytics 호출 경로 추가FirebaseAnalyticsServiceImpl추가 및 Firebase 이벤트 매핑 구성RootViewModel,MainViewModel에서login,home,today,profile화면 노출 추적 연결HomeViewModel,TodoListViewModel에서 Todo 생성 추적 연결TodoListViewModel,TodayViewModel에서 Todo 완료 추적 연결HomeViewModel에서 웹페이지 저장 성공 시webpage_create추적 연결PushNotificationOpenHandler추가를 통한 푸시 오픈 추적 및 라우팅 분리Info.plist에FirebaseAutomaticScreenReportingEnabled = NO추가로 자동 화면 추적 비활성화 적용DeleteWebPageTests보정 및DevLog.xcworkspace/DevLog스킴 빌드 검증 완료📸 영상 / 이미지 (Optional)