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

Analytics: Add reset time click tracking #464

Merged

Conversation

ksharma-xyz
Copy link
Owner

@ksharma-xyz ksharma-xyz commented Dec 18, 2024

TL;DR

Added analytics tracking for the reset date/time button in the trip planner.

What changed?

  • Created a new DateTimeSelectorEvent sealed interface for handling UI events
  • Added tracking for when users click the reset date/time button
  • Connected the reset button click in the UI to the ViewModel
  • Implemented analytics tracking through the Analytics interface

How to test?

  1. Open the trip planner
  2. Navigate to the date/time selector screen
  3. Click the reset button
  4. Verify that the analytics event ResetTimeClickEvent is being tracked

Why make this change?

To better understand how users interact with the date/time selector, specifically tracking when they reset their selections to the current time. This data will help inform future improvements to the trip planning experience.

@ksharma-xyz ksharma-xyz changed the title Add analytics events for DateTimeSelectorScreen Analytics: Add reset time click tracking Dec 18, 2024
@ksharma-xyz ksharma-xyz marked this pull request as ready for review December 18, 2024 07:17
@ksharma-xyz ksharma-xyz force-pushed the 12-17-add_click_events_for_timetablescreen branch from cee56a9 to 2bf5da2 Compare December 18, 2024 22:29
@ksharma-xyz ksharma-xyz force-pushed the 12-18-add_analytics_events_for_datetimeselectorscreen branch from 093ca2e to ffcf220 Compare December 18, 2024 22:29
@ksharma-xyz ksharma-xyz force-pushed the 12-17-add_click_events_for_timetablescreen branch from 2bf5da2 to a178b2f Compare December 19, 2024 07:37
Base automatically changed from 12-17-add_click_events_for_timetablescreen to main December 19, 2024 19:41
@ksharma-xyz ksharma-xyz force-pushed the 12-18-add_analytics_events_for_datetimeselectorscreen branch from ffcf220 to c454181 Compare December 19, 2024 19:41
@ksharma-xyz ksharma-xyz merged commit ccf1226 into main Dec 21, 2024
2 checks passed
@ksharma-xyz ksharma-xyz deleted the 12-18-add_analytics_events_for_datetimeselectorscreen branch December 21, 2024 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant