From ccd7e952cc26fe7a9b62a492687fa19c1578245b Mon Sep 17 00:00:00 2001
From: Patrick <patrick@failmail.dev>
Date: Fri, 19 Jul 2024 22:57:59 +0200
Subject: [PATCH] fix: only send email on demand fix: format mail as html

---
 src/mail.rs |  2 +-
 src/main.rs | 16 +++++++++-------
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/mail.rs b/src/mail.rs
index f4ec21d..da6a6a0 100644
--- a/src/mail.rs
+++ b/src/mail.rs
@@ -56,7 +56,7 @@ pub fn send_notification(
             "PR-tracker: {pr_number}: {pr_title} has reached {:?}",
             branches
         ))
-        .header(ContentType::TEXT_PLAIN)
+        .header(ContentType::TEXT_HTML)
         .body(body)
         .unwrap();
 
diff --git a/src/main.rs b/src/main.rs
index 07c0ee6..392ee91 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -201,13 +201,15 @@ async fn update_subscribers<S>(_request: Request<S>) -> http_types::Result<Respo
                         println!("{:#?}", val);
                         let to_do = &current - &val;
                         println!("They will be notified for: {:#?}", to_do);
-                        send_notification(
-                            &file_name,
-                            &to_do,
-                            page.pr_number.as_ref().unwrap(),
-                            page.pr_title.as_ref().unwrap(),
-                            !remaining,
-                        )?;
+                        if !to_do.is_empty() {
+                            send_notification(
+                                &file_name,
+                                &to_do,
+                                page.pr_number.as_ref().unwrap(),
+                                page.pr_title.as_ref().unwrap(),
+                                !remaining,
+                            )?;
+                        }
                     }
                 }
                 if !remaining {