Are there concerns with this PR? <Do not initializ...
# proj-agent-canvas
c
I'm not too sure why PostHog had that init before
h
@quick-window-91705 may be best to answer
w
this should be fine
fyi I believe
VITE_DO_NOT_TRACK
needs to be set a build time
so this may not have much effect?
c
I think it also fixes it at runtime when browser has Do Not Track
👍 1
w
hmm...we ideally would at least be able to register the initial "ping", so we know our opt-out rate for telemetry. I'm not sure what best practices are here
h
I think this just aligns with the docs as is in telemetry.ts github.com/OpenHands/OpenHands/blob/…/telemetry.ts#… . Which someone could always have done but now the VITE_DO_NOT_TRACK env will also work as written