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,
@@ -211,7 +212,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

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