Skip to content

[Feat] 화면 ID 도입 #357

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
PeraSite merged 11 commits into from
Sep 29, 2025
Merged

[Feat] 화면 ID 도입 #357

PeraSite merged 11 commits into from
Sep 29, 2025

Conversation

@PeraSite
Copy link
Member

@PeraSite PeraSite commented Sep 20, 2025 •
edited
Loading

Summary

iOS와 Android의 화면 ID 애널리틱스 통합을 목적으로 수정했습니다.

To reviewers

  • BaseActivity와 BaseFragment에 로깅하는 코드를 추가했는데, MainActivity와 CafeteriaFragment의 역할이 겹쳐shouldLogScreenId라는 함수를 특별히 도입했습니다. ScreenId.NONE을 추가하는 방안도 고려해보았으나 MainActivity가 뒤로가기 버튼을 누를 때마다 Resume되길래 이 방식으로 진행했습니다.
  • 터미널에서 adb shell setprop log.tag.FA VERBOSE, adb shell setprop log.tag.FA-SVC VERBOSE 명령어 2개를 입력하면 정확히 언제 Firebase Event가 서버로 전송되는지 알 수 있습니다. 중복된 screen_view 이벤트나 너무 짧은 시간 보인 화면은 알아서 제외하더라구요. 참고하시면 좋을 것 같습니다!
  • adb shell setprop debug.firebase.analytics.app com.eatssu.android.debug 를 입력하면 Firebase DebugView에서 실시간으로 이벤트를 볼 수 있는데 이상하게 제 에뮬레이터에서는 무슨 짓을 해도 디버그 기기로 등록이 안됩니다... 테스트 부탁드려요!
  • 다음 화면 ID는 특별한 설명이 필요합니다.
    • 점검 화면이 (아직)따로 존재하지 않는 LOGIN_MAINTENANCE
    • 내 정보 창에서 수정하게끔 바뀌어 필요 없는 LOGIN_NICKNAME
    • 안드로이드에서는 로그인이 필수라 필요 없는 REVIEW_V1_LOGIN_REQUIRED
    • "작성한 내 리뷰 수정" 페이지가 피그마에는 없어서 REVIEW_V1_MODIFY는 임의로 REV_001004_V1로 지정했습니다!
    • 리뷰 V2 관련 화면 ID는 추후 리뷰 V2 PR 머지되면 진행하겠습니다!
    • 리크루팅 페이지로 이동, 정책 등 WebViewActivity를 사용하는 부분에는 EXTERNAL_LINK 라는 화면 ID를 임의로 만들었습니다.

@PeraSite PeraSite self-assigned this Sep 20, 2025
@PeraSite PeraSite marked this pull request as ready for review September 20, 2025 12:21
HI-JIN2
HI-JIN2 approved these changes Sep 20, 2025
View reviewed changes
Copy link
Member

@HI-JIN2 HI-JIN2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고많으셨습니다!

디버그뷰에서 확인해보았습니다!

한가지 문제점은..... 컴포즈에서는 어떻게할지가 관건일 것 같아요... 컴포즈로 뷰를 구성할때에 BaseActivity를 만드는걸 못본 것 같아서... 찾아보고 함께 이야기 나눠봐야 할 것 같슴다

@PeraSite
Copy link
Member Author

PeraSite commented Sep 21, 2025

수고많으셨습니다!

디버그뷰에서 확인해보았습니다!

한가지 문제점은..... 컴포즈에서는 어떻게할지가 관건일 것 같아요... 컴포즈로 뷰를 구성할때에 BaseActivity를 만드는걸 못본 것 같아서... 찾아보고 함께 이야기 나눠봐야 할 것 같슴다

관련 내용을 까먹고 To reviewers에 작성하지 못했네요! 컴포즈에서는 LaunchedEffect(Unit) 를 사용해서 트래킹합니다. TrackScreenViewEvent 라는 Composable 하나를 모든 뷰에서 돌려쓰게끔 수정했어요!

@HI-JIN2
Copy link
Member

HI-JIN2 commented Sep 22, 2025

EXTERNAL_LINK은 각 ID에 맞게 수정되었으면 좋겠습니다.
단순 링크인지를 확인하려고 이벤트를 수집하는것은 아니니까요!

@PeraSite
Copy link
Member Author

PeraSite commented Sep 22, 2025

EXTERNAL_LINK은 각 ID에 맞게 수정되었으면 좋겠습니다. 단순 링크인지를 확인하려고 이벤트를 수집하는것은 아니니까요!

아요는 저희처럼 앱 내부 WebView에서 링크를 보여주는게 아니라 그냥 브라우저에서 링크를 열고 있는데, 저도 굳이 웹뷰로 보여주지 않고 그냥 아요처럼 바로 링크 이동시켜줘도 된다고 생각해요.
깃 로그 보니 24년 3월달에 진님께서 웹뷰를 도입하신 것 같은데... 당시에 외부 브라우저로 링크 이동은 고려 사항이 아니었을까요?

@HI-JIN2
Copy link
Member

HI-JIN2 commented Sep 23, 2025 •
edited
Loading

EXTERNAL_LINK은 각 ID에 맞게 수정되었으면 좋겠습니다. 단순 링크인지를 확인하려고 이벤트를 수집하는것은 아니니까요!

아요는 저희처럼 앱 내부 WebView에서 링크를 보여주는게 아니라 그냥 브라우저에서 링크를 열고 있는데, 저도 굳이 웹뷰로 보여주지 않고 그냥 아요처럼 바로 링크 이동시켜줘도 된다고 생각해요.
깃 로그 보니 24년 3월달에 진님께서 웹뷰를 도입하신 것 같은데... 당시에 외부 브라우저로 링크 이동은 고려 사항이 아니었을까요?

앱을 사용하다가 앱에서 나가지는게 자연스러운 액션은 아니라고 생각하였습니다.
저희가 제공하는 웹뷰가 웹에서 뭘 누르고 하는 것이 필요한게 아니라, 그냥 단순 뷰어의 역할만 요구하기 때문에 웹뷰가 적절하다 생각하였습니다. iOS가 웹뷰를 안띄우는건 몰랐네용 ㅎㅎ

@HI-JIN2
Copy link
Member

HI-JIN2 commented Sep 24, 2025

iOS 웹뷰인것으로 확인하였습니다. (topbar + 웹뷰) 안드아요 동일합니다

@PeraSite
feat: 외부 링크 각각 화면 ID 부여 및 적용
76f49ea
@PeraSite
feat: WebViewActivity 화면 ID 로깅 추가
5af5e66
@PeraSite
Merge branch 'develop' into feat/screen-id-analytics
b82174e 
# Conflicts:
#	app/src/main/java/com/eatssu/android/presentation/map/MapFragmentView.kt
#	app/src/main/java/com/eatssu/android/presentation/mypage/SignOutActivity.kt
@PeraSite
fix: IntroActivity 패키지 변경 반영
1dea5bd
@PeraSite PeraSite merged commit f4180db into develop Sep 29, 2025
1 check passed
@PeraSite PeraSite deleted the feat/screen-id-analytics branch September 29, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HI-JIN2 HI-JIN2 HI-JIN2 approved these changes

@kangyuri1114 kangyuri1114 Awaiting requested review from kangyuri1114

Assignees

@PeraSite PeraSite

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@PeraSite @HI-JIN2