-
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
EP-261/Android: Video Playback Started #1172
Conversation
# Conflicts: # app/src/main/java/com/kickstarter/libs/AnalyticEvents.kt # app/src/main/java/com/kickstarter/libs/utils/EventContextValues.kt # app/src/main/java/com/kickstarter/libs/utils/EventName.kt
@@ -0,0 +1,65 @@ | |||
package com.kickstarter.viewmodels |
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.
馃コ more migrations to Kotlin! 馃憦
@@ -475,6 +475,31 @@ class LakeTest : KSRobolectricTestCase() { | |||
this.lakeTest.assertValues("Project Page Pledge Button Clicked") | |||
} | |||
|
|||
@Test | |||
fun testVideoProperties() { |
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.
This is great @hadia is perfect! just a small thing, can you move this test from LakeTest
to SegmentTest
or duplicate it on SegmentTest
?, Lake client will be deleted some time in the near future alongside with the tests for it, so no real need unless some test fails on keep it updated
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.
sure
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.
馃弳 looks great!
馃摬 What
add Video Playback Started segment event
馃 Why
A user clicks the play button on the main video on a project page
馃洜 How
Add video Properties
Add Event Handler in Video ViewModel
Update project property "Launched_at" and Deadline to time stamp to solve violations
Add test
馃憖 See
馃搵 QA
open Project and click the video Play button
Story 馃摉
https://kickstarter.atlassian.net/browse/EP-261