-
Notifications
You must be signed in to change notification settings - Fork 45
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
[Bug 1883593] Instrumentation to capture more clicks via glean #2164
Conversation
- An empty string causes glean's data attributes to be present in Consumers that don't pass these props, causing clicks to be recorded in them - As per https://svelte.dev/docs/basic-markup#attributes-and-props, setting it to "null"
@rosahbruno I have added a new instrumentation in the code to capture clicks on links corresponding to Would it be possible for you to give me an early feedback on this instrumentation? I just want to make sure that I am heading in right direction as a lot of instrumentation will follow the same pattern. Thanks. |
- Capture clicks in "Access" section - Capture clicks on "Sampling Configuration Snippet" inside "Metric sampling" section
- Capture clicks on "Metrics", "Pings", "Tags", "Application IDs" and "Third-Party Data" tabs
Okay. I am marking this PR read to review. This PR adds instrumentation to capture clicks on:
I have tested this change locally and it works as expected. |
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 to me, with one minor question/nit about formatting in one place.
Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1883593
Pull Request checklist
fixes, if applicable)