You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Talking to Natalie, we should change those human readable page names into a more "id" like format, e.g. Connection.NewPage, instead of human readable ones, when fixing this issue. Also we should make sure, that unknown URLs will still trigger a page call. We could for them just use an ID UnknownPage, since segment already tracks the URL as an additional parameter anyway.
The text was updated successfully, but these errors were encountered:
Currently the segment calls for
page
(https://github.com/airbytehq/airbyte/blob/master/airbyte-webapp/src/hooks/services/Analytics/useTrackPageAnalytics.tsx#L15) on Cloud are not firing, since the relevant code is checking for a page URL and only firing if a page name is found, which currently doesn't ever work on Cloud due to workspace ids in the URL.Talking to Natalie, we should change those human readable page names into a more "id" like format, e.g.
Connection.NewPage
, instead of human readable ones, when fixing this issue. Also we should make sure, that unknown URLs will still trigger a page call. We could for them just use an IDUnknownPage
, since segment already tracks the URL as an additional parameter anyway.The text was updated successfully, but these errors were encountered: