-
Notifications
You must be signed in to change notification settings - Fork 15
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
Include Matomo-client for tracking default metrics #6535
Conversation
ac83c7b
to
7fdd931
Compare
7fdd931
to
9cf3c32
Compare
d627966
to
53bbb95
Compare
53bbb95
to
a37f511
Compare
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.
There are some small things that I'd like to refactor - but I'll do that in a separate PR because the implementation here looks very solid and thorough (I especially like the attention to detail in the .mjs files). So I'm merging as-is so hopefully we can tracking data and in the meantime do the refactor separately. (Why refactor? This question will be answered in the new PR).
AB#33700
Describe your changes
Checklist before requesting a review
Preview deployment
https://lively-river-04adce503-6535.westeurope.5.azurestaticapps.net