Skip to content
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.

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

Feat [#98] amplitude event 추가 #99

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

kim-seonwoo
Copy link
Contributor

🍀 작업 내용

앰플리튜드 이벤트 추적 추가

AmplitudeManager.shared.trackEvent("logout")

다음과 같이 앰플리튜드 추적 이벤트에 대해 추가했어요

🚀 PR Point

사실 이게 기존 코드에 넣기 애매한 구석이 있어서..
이벤트 트래커를 굴러가기만 하는 곳에 넣은 감이 있긴 해요...
예리한 위치 추천 부탁드립니다!

✚ 코드 리뷰 반영 사항

✅ CheckList

  • 오류 없이 빌드되는지 확인
  • 로그용 print문 제거
  • 불필요한 주석 제거
  • 코드 컨벤션 확인

🔗 Issue

Resolved #98

@kim-seonwoo kim-seonwoo added this to the Clody 3rd Sprint milestone Feb 27, 2025
@kim-seonwoo kim-seonwoo requested a review from Nya128 February 27, 2025 00:31
@kim-seonwoo kim-seonwoo self-assigned this Feb 27, 2025
@kim-seonwoo kim-seonwoo linked an issue Feb 27, 2025 that may be closed by this pull request
4 tasks
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.

[Feat] Amplitude 도입 (홈, 리스트, 일기작성, 알림)
1 participant