Merge pull request #1824 from zed-industries/change-telemetry-event-field-name-to-app

Joseph T. Lyons created 3 years ago

Change telemetry event field name to app

Change summary

crates/client/src/amplitude_telemetry.rs | 5 +++--
crates/client/src/telemetry.rs           | 5 +++--
2 files changed, 6 insertions(+), 4 deletions(-)

Detailed changes

crates/client/src/amplitude_telemetry.rs

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

crates/client/src/telemetry.rs

@@ -71,7 +71,8 @@ struct MixpanelEventProperties {
     os_version: Option<Arc<str>>,
     app_version: Option<Arc<str>>,
     signed_in: bool,
-    platform: &'static str,
+    #[serde(rename = "App")]
+    app: &'static str,
 }
 
 #[derive(Serialize)]
@@ -229,7 +230,7 @@ impl Telemetry {
                 os_version: state.os_version.clone(),
                 app_version: state.app_version.clone(),
                 signed_in: state.metrics_id.is_some(),
-                platform: "Zed",
+                app: "Zed",
             },
         };
         state.queue.push(event);