WIP

crates/config/src/server.rs

@@ -199,7 +199,7 @@ impl Notification {
             webhook: None,
         }
     }
-    pub const fn new_webhook(webhook: OutboundWebhook) -> Self {
+    pub const fn webhook(webhook: OutboundWebhook) -> Self {
         Self {
             r#type: NotificationType::Webhook,
             webhook: Some(webhook),
@@ -210,10 +210,6 @@ impl Notification {
         self.r#type
     }
 
-    pub const fn webhook(&self) -> Option<&OutboundWebhook> {
-        self.webhook.as_ref()
-    }
-
     pub fn webhook_url(&self) -> Option<String> {
         self.webhook.clone().map(|x| x.url)
     }

crates/config/src/tests.rs

@@ -751,7 +751,7 @@ mod given {
         ServerStorage::new(a_name().into())
     }
     pub fn a_notification_config() -> Notification {
-        Notification::new_webhook(an_outbound_webhook())
+        Notification::webhook(an_outbound_webhook())
     }
     pub fn some_forge_configs() -> BTreeMap<String, ForgeConfig> {
         [(a_name(), a_forge_config())].into()

crates/server-actor/src/handlers/notify_user.rs

@@ -22,7 +22,7 @@ impl Handler<NotifyUser> for ServerActor {
                 let timestamp = timestamp.unix_timestamp();
                 let to_sign = format!("{message_id}.{timestamp}.{payload}");
                 tracing::info!(?to_sign, "");
-                let Some(webhook) = notification.webhook() else {
+                let Some(webhook) = notification.webhook.as_ref() else {
                     tracing::warn!("Invalid notification configuration - can't sent notification");
                     return;
                 };