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

EP-421/Android: CTA Clicked (project) #1216

Merged
merged 16 commits into from
Apr 22, 2021
Merged

Conversation

hadia
Copy link
Contributor

@hadia hadia commented Apr 21, 2021

📲 What

send CTA Clicked (project)User taps on a project from the results on the search page.

🤔 Why

  1. Event Name -> CTA Clicked
  2. context_cta -> ​project
  3. context_page -> ​search
  4. context_location -> ​search_results
  5. context_type -> ​results

🛠 How

image
image

👀 See

Screen Shot 2021-04-21 at 3 49 15 PM

📋 QA

the open project as a search result

Story 📖

https://kickstarter.atlassian.net/browse/EP-421

@hadia hadia changed the title Hadia/ep 421 search project EP-421/Android: CTA Clicked (project) Apr 21, 2021
# Conflicts:
#	app/src/main/java/com/kickstarter/libs/AnalyticEvents.kt
#	app/src/test/java/com/kickstarter/libs/SegmentTest.kt
@hadia hadia marked this pull request as ready for review April 21, 2021 16:47
@leighdouglas leighdouglas self-assigned this Apr 21, 2021
@@ -749,6 +751,27 @@ class AnalyticEvents(trackingClients: List<TrackingClientType?>) {
client.track(CTA_CLICKED.eventName, props)
}

/**
* Tracks a discover project clicks on the search result.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add parameter documentation to this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done ✅

/**
* Tracks a discover project clicks on the search result.
*/
fun trackDiscoverSearchResultProjectCATClicked(
Copy link
Contributor

Choose a reason for hiding this comment

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

please correct the spelling of this method -> trackDiscoverSearchResultProjectCTAClicked

Copy link
Contributor

@leighdouglas leighdouglas left a comment

Choose a reason for hiding this comment

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

Please address the comments and it's good to go! 🎉

Copy link
Contributor

@Arkariang Arkariang left a comment

Choose a reason for hiding this comment

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

lgtm

@hadia hadia merged commit 9a8a38f into master Apr 22, 2021
@hadia hadia deleted the hadia/EP_421_search_project branch April 22, 2021 12:33
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