ui telemetry for all events - #9783
Conversation
| // 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; |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Checklist: