chore: Switch Apprise to markdown (alexjustesen#2531)

Author: svenvg93

app/Listeners/ProcessCompletedSpeedtest.php

@@ -88,7 +88,7 @@ private function notifyAppriseChannels(Result $result): void
             }
 
             Notification::route('apprise_urls', $channelUrl)
-                ->notify(new SpeedtestNotification($title, $body, 'info'));
+                ->notify(new SpeedtestNotification($title, $body, 'info', 'markdown'));
         }
     }
 

app/Listeners/ProcessUnhealthySpeedtest.php

@@ -110,7 +110,7 @@ private function notifyAppriseChannels(Result $result): void
             }
 
             Notification::route('apprise_urls', $channelUrl)
-                ->notify(new SpeedtestNotification($title, $body, 'warning'));
+                ->notify(new SpeedtestNotification($title, $body, 'warning', 'markdown'));
         }
     }
 

app/Notifications/Apprise/SpeedtestNotification.php

@@ -14,6 +14,7 @@ public function __construct(
         public string $title,
         public string $body,
         public string $type = 'info',
+        public string $format = 'markdown',
     ) {}
 
     /**
@@ -35,6 +36,7 @@ public function toApprise(object $notifiable): AppriseMessage
             ->urls($notifiable->routes['apprise_urls'])
             ->title($this->title)
             ->body($this->body)
-            ->type($this->type);
+            ->type($this->type)
+            ->format($this->format);
     }
 }

app/Notifications/Apprise/TestNotification.php

@@ -25,10 +25,14 @@ public function via(object $notifiable): array
      */
     public function toApprise(object $notifiable): AppriseMessage
     {
+        $body = '👋 This is a test notification from **'.config('app.name')."**.\n\n";
+        $body .= "If you're seeing this, your Apprise notification channel is configured correctly!\n\n";
+
         return AppriseMessage::create()
             ->urls($notifiable->routes['apprise_urls'])
             ->title('Test Notification')
-            ->body('👋 Testing the Apprise notification channel.')
-            ->type('info');
+            ->body($body)
+            ->type('info')
+            ->format('markdown');
     }
 }

resources/views/apprise/speedtest-completed.blade.php

@@ -1,11 +1,13 @@
-A new speedtest on {{ config('app.name') }} was completed using {{ $service }}.
+A new speedtest on **{{ config('app.name') }}** was completed using **{{ $service }}**.
 
-Server name: {{ $serverName }}
-Server ID: {{ $serverId }}
-ISP: {{ $isp }}
-Ping: {{ $ping }}
-Download: {{ $download }}
-Upload: {{ $upload }}
-Packet Loss: {{ $packetLoss }} %
-Ookla Speedtest: {{ $speedtest_url }}
-URL: {{ $url }}
+### Results
+- **Server:** {{ $serverName }} (ID: {{ $serverId }})
+- **ISP:** {{ $isp }}
+- **Ping:** {{ $ping }}
+- **Download:** {{ $download }}
+- **Upload:** {{ $upload }}
+- **Packet Loss:** {{ $packetLoss }}%
+
+### Links
+- [View Ookla Results]({{ $speedtest_url }})
+- [View Dashboard]({{ $url }})

resources/views/apprise/speedtest-threshold.blade.php

@@ -1,7 +1,12 @@
-A new speedtest on **{{ config('app.name') }}** was completed using **{{ $service }}** on **{{ $isp }}** but a threshold was breached.
-
+A new speedtest on **{{ config('app.name') }}** was completed using **{{ $service }}** on **{{ $isp }}** but a threshold was breached
+### Failed Metrics
 @foreach ($metrics as $item)
-- **{{ $item['name'] }}** {{ $item['threshold'] }}: {{ $item['value'] }}
+- **{{ $item['name'] }}**
+  - **Threshold:** {{ $item['threshold'] }} | **Actual:** {{ $item['value'] }}
 @endforeach
-- **Ookla Speedtest:** {{ $speedtest_url }}
-- **URL:** {{ $url }}
+### Server Information
+- **Server:** {{ $serverName }} (ID: {{ $serverId }})
+- **ISP:** {{ $isp }}
+### Links
+- [View Ookla Results]({{ $speedtest_url }})
+- [View Dashboard]({{ $url }})