From 495fd151f5794d2d225bd918c687a3f465eeebcb Mon Sep 17 00:00:00 2001 From: Joseph T Lyons Date: Mon, 31 Oct 2022 19:18:03 -0400 Subject: [PATCH] Revert Amplitude's "app" name back to "platform" 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. --- crates/client/src/amplitude_telemetry.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/crates/client/src/amplitude_telemetry.rs b/crates/client/src/amplitude_telemetry.rs index 4883124d28a79fbefc0243b160766d7767634f61..de17ae6e8d69e20de0053cb4a8967739572ca236 100644 --- a/crates/client/src/amplitude_telemetry.rs +++ b/crates/client/src/amplitude_telemetry.rs @@ -70,8 +70,7 @@ struct AmplitudeEvent { os_name: &'static str, os_version: Option>, app_version: Option>, - #[serde(rename = "App")] - app: &'static str, + platform: &'static str, release_channel: Option<&'static str>, event_id: usize, session_id: u128, @@ -221,7 +220,7 @@ impl AmplitudeTelemetry { user_id: state.metrics_id.clone(), device_id: state.device_id.clone(), os_name: state.os_name, - app: "Zed", + platform: "Zed", os_version: state.os_version.clone(), app_version: state.app_version.clone(), release_channel: state.release_channel,