-
Notifications
You must be signed in to change notification settings - Fork 9
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
[Feature/#908] 오늘의 솝마디 앰플리튜드 & NDS & 에러뷰 적용 #910
Conversation
- di 모듈 추가 - network api 구현
- mapper 구현 - 레포지토리 인터페이스 생성 및 적용 - 힐트모듈 수정
…into feature/869
…into feature/877
…into feature/877
|
val amplitudeTracker = LocalAmplitudeTracker.current.also { | ||
it.track( | ||
type = EventType.VIEW, | ||
name = "view_soptmadi_charmcard", | ||
) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이렇게 두면 state가 성공한 경우가 아니어도 무조건 찍히지 않나?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오? 성공 실패는 생각도 못해본 관점이긴한데 그럼 fortuneDetail쪽은 어느쪽에서 해줘야할까요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
나라면FortuneAmuletScreen
내부에서 트래커 뷰 찍을듯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 수정좀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다.
private var isAnonymous: Boolean = false | ||
private var userId = DEFAULT_ID | ||
|
||
init { | ||
refresh() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fetch 정도는 어떠신가요? 처음 데이터 불러올때에도 refresh라고 하나...?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fetch는 이제.. 리모트데이터 소스 느낌 ㅋ.. updateUi로..해볼게요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
에잉 워낙 잘해서 코리 달것도 없네용~ ㅋㅋ
amplitudeTracker.track( | ||
type = CLICK, | ||
name = "uncheck_anonymity", | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
가물가물한디 이거 수집 목적이 익명 해제를 얼마나 하는지 였던거 같은데, 여기 두면 익명 해제 및 익명 재설정 모두 트래커 찍히지 않나용? 아니었다면 지송함다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
누를 때마다 true false 주기로 했심더
What is this issue?
Reference
KakaoTalk_Video_2024-10-14-22-00-17.mp4
To Reviewer