zed-industries/zed
1
1
Fork 0
mirror of https://github.com/zed-industries/zed.git synced 2024-11-09 21:26:14 +03:00
Code Issues Packages Projects Releases Wiki Activity

Revert Amplitude's "app" name back to "platform"

Browse Source 
This was unintentional.  We only want to rename the Mixpanel telemetry "platform" field to "app."  We want to keep it as "platform" on Amplitude because we want to keep using Amplitude for a bit, and the event fields should be the same.
This commit is contained in:
Joseph T Lyons 2022-10-31 19:18:03 -04:00
parent 482a5bb02a
commit 495fd151f5
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
crates/client/src/amplitude_telemetry.rs
View File

@ -70,8 +70,7 @@ struct AmplitudeEvent {
os_name: &'static str, os_name: &'static str,
os_version: Option<Arc<str>>, os_version: Option<Arc<str>>,
app_version: Option<Arc<str>>, app_version: Option<Arc<str>>,
#[serde(rename = "App")] platform: &'static str,
app: &'static str,
release_channel: Option<&'static str>, release_channel: Option<&'static str>,
event_id: usize, event_id: usize,
session_id: u128, session_id: u128,
@ -221,7 +220,7 @@ impl AmplitudeTelemetry {
user_id: state.metrics_id.clone(), user_id: state.metrics_id.clone(),
device_id: state.device_id.clone(), device_id: state.device_id.clone(),
os_name: state.os_name, os_name: state.os_name,
app: "Zed", platform: "Zed",
os_version: state.os_version.clone(), os_version: state.os_version.clone(),
app_version: state.app_version.clone(), app_version: state.app_version.clone(),
release_channel: state.release_channel, release_channel: state.release_channel,