[fix] 비로그인 시에도 설정 화면 접속 가능하도록 수정#1543
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Walkthrough
Changes프로필 설정 이동
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
| } else { | ||
| navigator.navigateToSignIn(context, DEEPLINK_MAIN_PROFILE).let { context.startActivity(it) } | ||
| if (!uiState.isLoggedIn) { | ||
| EventLogger.logCampusClickEvent(AnalyticsConstant.Label.LOGIN_PROMPT, "설정(비로그인)") |
There was a problem hiding this comment.
[Minor] LOGIN_PROMPT 레이블 의미 불일치
LOGIN_PROMPT라는 레이블은 로그인 유도 팝업이나 안내가 표시될 때 사용하는 이름처럼 읽힙니다. 그러나 이 PR의 목적 자체가 "로그인 없이도 설정 화면 진입 허용"이므로, 실제로는 로그인 유도 없이 설정 화면으로 바로 이동합니다.
분석 결과를 오해 없이 추적하려면 레이블을 행동을 정확히 표현하는 이름으로 변경하는 것이 좋습니다.
| EventLogger.logCampusClickEvent(AnalyticsConstant.Label.LOGIN_PROMPT, "설정(비로그인)") | |
| EventLogger.logCampusClickEvent(AnalyticsConstant.Label.SETTINGS_WITHOUT_LOGIN, "설정(비로그인)") |
혹은 기존 AnalyticsConstant 규칙에 맞게 Profile.HOME_SETTINGS_WITHOUT_LOGIN 같은 네이밍도 고려해 볼 수 있습니다. 분석팀과 레이블 명세를 먼저 조율해 주세요.
There was a problem hiding this comment.
명세에는 없었지만 이전에 프로필 모듈을 분리하면서 있었던 로깅이라 우선 추가했습니다!
There was a problem hiding this comment.
커밋 히스토리 확인 결과 제거하는 것이 맞는 것 같아서 제거하겠습니다.
| @@ -76,11 +76,10 @@ fun ProfileScreen( | |||
| }, | |||
| onSettingClick = { | |||
| EventLogger.logCampusClickEvent(AnalyticsConstant.Label.Profile.HOME_SETTINGS, "설정") | |||
There was a problem hiding this comment.
[Info] 비로그인 사용자의 이벤트 이중 발화 — 의도적인지 확인 필요
비로그인 사용자가 설정을 탭하면 아래 두 이벤트가 순차적으로 발화됩니다:
HOME_SETTINGS("설정") — 78번째 줄LOGIN_PROMPT("설정(비로그인)") — 80번째 줄
HOME_SETTINGS는 로그인 여부와 무관하게 모든 탭에서 발화하므로 의도된 동작이라면 문제없습니다. 다만 분석 대시보드에서 "설정 클릭 수"를 집계할 때 비로그인 탭이 두 이벤트에 모두 포함된다는 점을 분석팀이 인지하고 있어야 합니다.
만약 비로그인 상태를 나타내는 이벤트 하나만 발화하면 충분하다면, 아래처럼 조건 안에만 발화시키는 구조도 고려해 볼 수 있습니다.
onSettingClick = {
if (uiState.isLoggedIn) {
EventLogger.logCampusClickEvent(AnalyticsConstant.Label.Profile.HOME_SETTINGS, "설정")
} else {
EventLogger.logCampusClickEvent(AnalyticsConstant.Label.LOGIN_PROMPT, "설정(비로그인)")
}
onNavigateToSetting()
}현재 이중 발화 방식이 의도된 설계라면 코드 그대로 유지해도 좋습니다. 분석팀과 합의된 내용인지 확인만 부탁드립니다.
bcsd-b-bot
Bot
added
ci:passed
and removed
ci:passed
coderabbit:complete
claude:complete
labels
|
2 participants
PR 개요
PR 체크리스트
작업사항
작업사항의 상세한 설명
논의 사항
스크린샷
추가내용
Summary by CodeRabbit