8000 [🛤] NT-936 Creator Details Clicked event by eoji · Pull Request #786 · kickstarter/android-oss · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[🛤] NT-936 Creator Details Clicked event #786

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

Merged
merged 14 commits into from
Mar 16, 2020
Merged

[🛤] NT-936 Creator Details Clicked event #786

merged 14 commits into from
Mar 16, 2020

Conversation

eoji
Copy link
Contributor
@eoji eoji commented Mar 11, 2020

📲 What

Added Creator Details Clicked event for Optimizely.

🤔 Why

So we can get those sweet funnel numbers.

🛠 How

  • Added OptimizelyEvent.CREATOR_DETAILS_CLICKED with value "Creator Details Clicked"
  • Tracking CREATOR_DETAILS_CLICKED when the creator details are clicked in ProjectViewModel.
  • Added tests to assert CREATOR_DETAILS_CLICKED is tracked when the button is clicked on live and not backed projects only.

👀 See

Nothing to see.

📋 QA

Check your local logs!
Check your data dog logs!

Story 📖

NT-936


fullProjectDataAndCurrentUser
.compose<Pair<ProjectData, User?>>(takeWhen(creatorInfoClicked))
.filter { it.first.project().isLive && !it.first.project().isBacking }

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we not show the creator info if you've already backed the project? 🤔

Copy link
Contributor Author
@eoji eoji Mar 16, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is only for tracking. I'm only tracking this for live not backed projects. Otherwise, I think the experiment would have really low conversion because it would fire for ended and already backed projects? But not sure because the experiment process has been pretty confusing.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok that makes sense!

@eoji eoji merged commit ed38201 into master Mar 16, 2020
@eoji eoji deleted the NT-936 branch March 16, 2020 18:23
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
0