[FEAT/#1645] 앰플리튜드 개편 - 3 - #1648
Merged
Merged
Conversation
- `UserStatus`를 Amplitude `view_type` 문자열로 변환하는 확장 함수에 KDoc 추가 - `String?` 타입을 `view_type`으로 변환하는 확장 함수 추가 (기본값 `visitor`)
- Update `tracker.trackViewType` calls to use `userStatus.toViewType()` instead of `userStatus.value`.
- `PokeAnalytics`를 추가하여 이벤트 명세(View/Click), 프로퍼티 키, 클릭 소스 등을 정의하고 관리하도록 개선함. - `Tracker.trackViewType` 확장 함수를 사용하여 기존의 하드코딩된 트래킹 로직을 리팩토링함. - `MessageListBottomSheetFragment`에 분석용 `viewType` 전달 및 익명 체크박스 클릭, 메시지 전송 클릭 이벤트 트래킹을 추가함. - `MessageItemClickListener`에서 메시지 객체 전체를 전달하도록 변경하여 `messageId` 로깅이 가능하도록 수정함. - `FriendListDetail`, `PokeNotification`, `PokeMain` 등 주요 화면에서의 클릭 소스(onboarding, main_alarm, friend_summary 등) 구분을 구체화함.
- Remove `mission.isCompleted` check before calling `onMissionItemClickTricked()` in `MissionComponent`.
- `NotificationAnalyticsEvent`, `NotificationLinkType` 등 알림 관련 분석 이벤트 및 프로퍼티 정의 - `EventType`에 `RECEIVED` 타입 추가 - 알림 수신(`RECEIVED_PUSH`), 푸시 클릭(`CLICK_PUSH`), 목록 진입, 전체 읽기, 상세 링크 클릭 등 주요 액션 트래킹 로직 구현 - 트래킹 시 `viewType` 식별을 위해 알림 관련 Activity 및 네비게이터에 `UserStatus` 전달 로직 추가 - `SchemeActivity`에서 앱 실행 상태(Cold/Warm start)에 따른 트래킹 처리 추가
- `SoptFirebaseMessagingService` 및 `SchemeActivity` 내 분석 이벤트 전송 시 `suspendRunCatching`을 적용하여 예외 처리 추가 - `SchemeActivity.Argument`에서 `isPush` 필드를 제거하고, `Intent`의 별도 Extra로 전달하도록 변경 - 사용자 상태 조회 실패 시 기본값(`UNAUTHENTICATED`)을 사용하도록 보완
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
2 participants
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.
Related issue 🛠
Work Description ✏️
notification_id,notification_link_type,notification_launch_type프로퍼티가 함께 전송되도록 구현.UserStatus를 Amplitudeview_type값으로 변환하는 로직을 공통화Screenshot 📸
N/A
Uncompleted Tasks 😅
To Reviewers 📢
앰플리튜드 개편 마무리 작업입니다.
아래 부분 위주로 확인 부탁드립니다.
view_type,notification_link_type,poke_click_source값이 의도한 케이스에 맞게 전달되는지