Add telemetry tracking of errors

firmware/src/telemetry.rs

@@ -2,6 +2,7 @@ use crate::http::Http;
 use crate::mixpanel::{self, Mixpanel};
 use crate::{article, json, nfc, user};
 use alloc::collections::VecDeque;
+use alloc::string::String;
 use embassy_time::{Duration, Instant};
 use embedded_io_async::Write;
 use log::{debug, info, warn};
@@ -28,6 +29,8 @@ pub enum Event {
     UserAuthenticated(user::UserId, nfc::Uid),
     /// Article purchased (user id, article id, amount, total price)
     ArticlePurchased(user::UserId, article::ArticleId, f32, f32),
+    /// Error occured (optional user id, error message)
+    Error(Option<user::UserId>, String),
 }
 
 impl Event {
@@ -39,18 +42,20 @@ impl Event {
             Event::AuthenticationFailed(..) => "authentication_failed",
             Event::UserAuthenticated(..) => "user_authenticated",
             Event::ArticlePurchased(..) => "article_purchased",
+            Event::Error(..) => "error",
         }
     }
 
     /// User id associated with this event, if any
-    pub fn user_id(&self) -> Option<&user::UserId> {
+    pub fn user_id(&self) -> Option<user::UserId> {
         #[allow(clippy::match_same_arms)]
         match self {
             Event::SystemStart => None,
             Event::DataRefreshed(..) => None,
             Event::AuthenticationFailed(..) => None,
-            Event::UserAuthenticated(user_id, ..) => Some(user_id),
-            Event::ArticlePurchased(user_id, ..) => Some(user_id),
+            Event::UserAuthenticated(user_id, ..) => Some(*user_id),
+            Event::ArticlePurchased(user_id, ..) => Some(*user_id),
+            Event::Error(user_id, ..) => *user_id,
         }
     }
 
@@ -85,6 +90,9 @@ impl Event {
                     .field("total_price", total_price)
                     .await?;
             }
+            Event::Error(_user_id, message) => {
+                object.field("error_message", message).await?;
+            }
         }
         Ok(())
     }

firmware/src/ui.rs

@@ -11,7 +11,7 @@ use crate::telemetry::{Event, Telemetry};
 use crate::user::{UserId, Users};
 use crate::vereinsflieger::Vereinsflieger;
 use crate::wifi::Wifi;
-use alloc::string::String;
+use alloc::string::{String, ToString};
 use core::convert::Infallible;
 use embassy_futures::select::{select, Either};
 use embassy_time::{with_timeout, Duration, TimeoutError, Timer};
@@ -116,6 +116,9 @@ impl<'a, RNG: RngCore, I2C: I2c, IRQ: Wait<Error = Infallible>> Ui<'a, RNG, I2C,
             let _ = self.buzzer.error().await;
         }
 
+        self.telemetry
+            .track(Event::Error(error.user_id(), error.to_string()));
+
         // Wait at least 1s without responding to keypad
         let min_time = Duration::from_secs(1);
         Timer::after(min_time).await;