Skip to content

[Feature] 프로필 로깅 추가#1537

Merged
TTRR1007 merged 2 commits into
developfrom
feature/#1496-profile-logging
Jul 19, 2026
Merged

[Feature] 프로필 로깅 추가#1537
TTRR1007 merged 2 commits into
developfrom
feature/#1496-profile-logging

Conversation

@TTRR1007

@TTRR1007 TTRR1007 commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

PR 개요

이슈 번호:

PR 체크리스트

  • Code convention을 잘 지켰나요?
  • Lint check를 수행하였나요?
  • Assignees를 추가했나요?

작업사항

  • 버그 수정
  • 신규 기능
  • 코드 스타일 수정 (포맷팅 등)
  • 리팩토링 (기능 수정 X, API 수정 X)
  • 기타

작업사항의 상세한 설명

프로필 로깅을 추가했습니다

논의 사항

스크린샷

추가내용

  • develop, sprint 브랜치를 향하고 있습니다
  • production 브랜치를 향하고 있습니다

Summary by CodeRabbit

  • 개선 사항
    • 프로필 화면의 로그인/로그아웃/설정/내 시간표 활동이 프로필 전용 분석 라벨로 기록됩니다.
    • 로그인 상태에 따라 로그인·로그아웃 처리와 분석 기록이 구분되어 더 정확하게 집계됩니다.

@TTRR1007
TTRR1007 requested a review from a team as a code owner July 18, 2026 14:35
@TTRR1007 TTRR1007 self-assigned this Jul 18, 2026
@coderabbitai

coderabbitai Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

프로필 전용 분석 라벨 4개를 추가하고, 프로필 화면의 로그인·로그아웃·설정·시간표 클릭 처리와 시간표 화면 연결을 변경했습니다.

Changes

프로필 화면 상호작용

Layer / File(s) Summary
프로필 분석 라벨 추가
core/analytics/.../AnalyticsConstant.kt
시간표, 로그인, 로그아웃, 설정을 식별하는 분석 상수를 추가했습니다.
클릭 처리 및 시간표 연결
feature/profile/.../ProfileScreen.kt
인증·설정 클릭에 프로필 라벨을 적용하고, 시간표 클릭 시 로그인 상태에 따른 화면 이동과 익명 여부를 전달하도록 연결했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • BCSDLab/KOIN_ANDROID#1531: 동일한 ProfileScreen.kt의 설정·시간표 클릭 동작을 수정해 코드 충돌 및 흐름 연관성이 있습니다.

Suggested reviewers: kym-p, kongwoojin, jaeyoung290

🚥 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 Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title 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 feature/#1496-profile-logging

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.

@bcsd-b-bot bcsd-b-bot Bot added the ci:passed label Jul 18, 2026
@sonarqubecloud

Copy link
Copy Markdown

@bcsd-b-bot bcsd-b-bot Bot added the ci:passed label Jul 18, 2026
@TTRR1007
TTRR1007 merged commit 510e121 into develop Jul 19, 2026
13 checks passed
@TTRR1007
TTRR1007 deleted the feature/#1496-profile-logging branch July 19, 2026 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants