search

appsmithorg / appsmith

Platform to build admin panels, internal tools, and dashboards. Integrates with 25+ databases and any API.
https://www.appsmith.com
Apache License 2.0
33.93k stars 3.66k forks source link

[Task]: Send mixpanel id as a header in every API call for anonymous users #19105

Closed eco-monk closed 1 year ago

eco-monk commented 1 year ago

Is there an existing issue for this?

SubTasks

This is to help with backend telemetry events not being fired for anonymous users.

eco-monk commented 1 year ago

Setup segment locally. And tested id being sent after a few API calls. Getting feedback now. image

eco-monk commented 1 year ago

Need to investigate some initial API's not getting the header

eco-monk commented 1 year ago

Header added. Need to handle on page load API's missing the header. (being discussed)

image
eco-monk commented 1 year ago

On page load actions will wait for analytics to be initialized. (on ready -> promise.resolve) Need to handle handle load error for analytics. (call promise.reject on load error)

Otherwise on page load actions will keep waiting indefinitely. Conversation started

eco-monk commented 1 year ago

Using cached ID from local storage instead.