-
Notifications
You must be signed in to change notification settings - Fork 31
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
Track chat routes on Plausible #713
Conversation
The Plausible bits look good. For every route, we'll need to manually add a custom event/goal in the Plausible dashboard. It looks like there's 8 routes currently? If you feel this will be burdensome in future (e.g. if we're adding lots of routes, having to remember to add the custom events each time etc.) then it may be worth considering just having a single Do we need the |
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 good for now.
@KevinEtchells given that we sometimes add new routes for testing, I'm tempted to use the single 'route' approach you mentioned above The difference between the |
@@ -8433,7 +8433,6 @@ body:has(.iai-environment-warning) main { | |||
.iai-chat-bubble__route { | |||
font-size: 0.625rem; | |||
line-height: 0.825rem; | |||
display: none; |
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 was removed as part of #698 and I think might have got re-added during a rebase
Context
This allows us to track the route used in a chat message on Plausible in streamed chat.
Changes proposed in this pull request
Sends the route to the JS app for non-staff users in a new 'hidden-route' event type (this can be removed when the route is available to all users).
For both the 'route' and 'hidden-route' event types, trigger a custom
Chat-message-route-[ROUTE NAME]
event in Plausible:https://plausible.io/docs/custom-event-goals#trigger-custom-events-manually-with-a-javascript-function
Guidance to review
Relevant links
https://technologyprogramme.atlassian.net/browse/REDBOX-439
https://plausible.io/docs/custom-event-goals#trigger-custom-events-manually-with-a-javascript-function
Things to check