-
Notifications
You must be signed in to change notification settings - Fork 992
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
Creator tools tracking #213
Conversation
-added ref tag for messages -added events for `Viewed Project Dashboard`, `Opened Project Switcher` and `Switched Projects`
@@ -51,6 +51,10 @@ public ViewModel(final @NonNull Environment environment) { | |||
.compose(bindToLifecycle()) | |||
.subscribe(this.projectAndStats); | |||
|
|||
this.projectsMenuClick |
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.
Could we rename this to projectList
instead? It would make more sense on the context of the app.
.take(1) | ||
.compose(bindToLifecycle()) | ||
.subscribe(__ -> this.koala.trackViewedMailbox(Mailbox.INBOX, null)); | ||
.subscribe(data -> this.koala.trackViewedMailbox(Mailbox.INBOX, data.first, data.second)); |
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.
Could you rename this for project and refTag instead of data?
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.
Nice job! 🥇 Just a few comments.
-renaming
-forgot to rename one more thing
* Creator tools tracking/instrumentation! -added ref tag for messages -added events for `Viewed Project Dashboard`, `Opened Project Switcher` and `Switched Projects` * updating tests * PR feedback: -renaming
Creator tools tracking/instrumentation!
-refactoring
CreatorDashboardViewModel
and tests-added ref tag for messages
-added events for
Viewed Project Dashboard
,Opened Project Switcher
andSwitched Projects