telemetry/add kedro_Viz_version to heaps

kedro-org / kedro-viz

Visualise your Kedro data and machine-learning pipelines and track your experiments.

https://demo.kedro.org

Apache License 2.0

680 stars 113 forks source link

telemetry/add kedro_Viz_version to heaps #2194

Closed Huongg closed 3 days ago

Huongg commented 1 week ago

Description

Fixes https://github.com/kedro-org/kedro-plugins/issues/923

To include kedro_viz_version when user interacts with viz

Development notes

When test this locally, ensure you enable consent: true in project-demo telemetry, and in the heap dash board development environment, it should show kedro_viz_version as one of the property Screenshot 2024-11-15 at 13 17 39

Checklist

[ ] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

rashidakanchwala commented 4 days ago

i cannot checkout this branch @jitu5 , can you ??

jitu5 commented 4 days ago

i cannot checkout this branch @jitu5 , can you ??

@rashidakanchwala Yes I can.

jitu5 commented 4 days ago

Hey @Huongg

I tested locally and here as well https://heapanalytics.com/app/env/2388822444/data-galaxy?view=live-data-feed but I cant see kedro-viz version on heap data for both CLI and UI event Screenshot 2024-11-18 at 1 01 29 p m

rashidakanchwala commented 4 days ago

@jitu5 , u r on prod environment, u should check dev environment

Huongg commented 4 days ago

hey @jitu5 let me know if you can see it in the development environment. Im happy to jump on the call to test this together. This is what I currently see from my side

Huongg commented 3 days ago

hey both I just pushed the change and tested again and its working from my side. Just tested with @rashidakanchwala too, can you confirm its okay now?