Skip to content

ui telemetry for all events - #9783

Open
pjain1 wants to merge 2 commits into
mainfrom
ui_instrumentation
Open

ui telemetry for all events#9783
pjain1 wants to merge 2 commits into
mainfrom
ui_instrumentation

Conversation

@pjain1

@pjain1 pjain1 commented Aug 3, 2026

Copy link
Copy Markdown
Member

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@pjain1 pjain1 changed the title enable ui telemetry for all events ui telemetry for all events Aug 3, 2026
// The URL the event was fired from.
// This is what lets events be attributed to the dashboard or canvas the user was on, since the
// resource name and the dashboard state (time range, filters, view mode) only exist in the URL.
page_url: string;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Since we are guarding against path not search params in layout, time range, filters etc might not get logged.

If we want url params then we need to remove the guard from layout around url path. But this will lead to explosion of events since url changes quite often, we could optimize debouncing by say 1sec, but might still be too much.

If we only want explore/canvas names then how about emitting just that? org, project_id, resourceName, resourceType might give full picture.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I am not concerned about event volume, I think we can handle it. At some point it would be good to have info for at least most used time ranges, dimensions and measures but semantics is the issue here loading a dimension on dashboard may not mean it is being used, if its filtered on, expanded, sorted by etc. then it may counted as used and I think we may need it for canvas as well not just explore.

So may be we start simple for now with just resource name and type and then later add these things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants