-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Checkout event improvements #813
Conversation
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 looking good! I think we could a few more tests to make sure we've covered everything.
Library/Koala/KoalaTests.swift
Outdated
@@ -475,7 +475,7 @@ final class KoalaTests: TestCase { | |||
|
|||
let koala = Koala(client: client, loggedInUser: loggedInUser) | |||
|
|||
koala.trackBackThisButtonClicked(project: project, screen: .projectPage) | |||
koala.trackBackThisButtonClicked(stateType: .pledge, project: project, screen: .projectPage) |
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.
Can we add tests for the other cases of PledgeStateCTAType
?
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.
Yes, will do!
@@ -167,4 +169,19 @@ internal final class PledgeCTAContainerViewViewModelTests: TestCase { | |||
self.vm.inputs.configureWith(value: (.right(.couldNotParseJSON), false)) | |||
self.pledgeRetryButtonIsHidden.assertValues([true, false]) | |||
} | |||
|
|||
func testNotifyDelegateCTATapped() { |
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.
Can we add a test to verify that tapping the button triggers the events in Koala?
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.
Yup, will do!
π² What
Improvements to monitoring user behavior for v1 checkout.
π€ Why
We now have events for:
Select Reward Button Clicked
Manage Pledge Button Clicked
View Rewards Button Clicked
View Your Pledge Button Clicked
Fix Pledge Button Clicked
We also renamed
Project Page Viewed
event toProject Page
β Acceptance criteria
KOALA_TRACKING
environment variable in the scheme. Then launch the app and navigate to a project. You should see theBack this project
CTA, tap it. You should see an event -"Back this Project Button Clicked"
in the console.Manage
CTA, tap it. You should see an event"Manage Pledge Button Clicked"
in the console.View Your Pledge
CTA, tap it. You should see an event"View your pledge Button Clicked"
in the console.View rewards
CTA, tap it. You should see an event"View Your Pledge Button Clicked"
in the console.