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

feat(web): Record time slider interactions with plausible #6707

Merged
merged 4 commits into from May 2, 2024

Conversation

tonypls
Copy link
Collaborator

@tonypls tonypls commented May 1, 2024

Issue

We don't know how users interact with out time slider

Description

This PR sends a custom event to Plausible for interactions with the time slider component, I've alos created a goal and custom property to record the unique interactions with the time slider and property of selected index.

Copy link
Contributor

@silkeholmebonnen silkeholmebonnen left a comment

Choose a reason for hiding this comment

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

Looks good, just one comment:)

onValueChange={(value) => onChange(value[0])}
onValueChange={(value) => {
onChange(value[0]);
trackTimeSliderEvent(value[0]);
Copy link
Contributor

Choose a reason for hiding this comment

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

As far as I can tell value[0] is just a number, e.g. for hourly its 0-24 and monthly 0-12, but how do we know if that number corresponds to hourly, daily, monthly or yearly? Or are we not interested in that?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good point, I think we should include the time aggregate too. I'll update the PR to include it

Copy link
Contributor

@silkeholmebonnen silkeholmebonnen left a comment

Choose a reason for hiding this comment

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

LGTM 🌐

@tonypls tonypls enabled auto-merge (squash) May 2, 2024 14:17
@tonypls tonypls merged commit 6421e87 into master May 2, 2024
21 checks passed
@tonypls tonypls deleted the tonyvanswet/avo-227-track-historical-slider branch May 2, 2024 14:21
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.

None yet

2 participants