From aaa9cb0a1eb71f69dd0bdd649e05a4aff103bb74 Mon Sep 17 00:00:00 2001
From: Gabriel Kaszewski <gabrielkaszewski@gmail.com>
Date: Fri, 8 May 2026 13:12:56 +0200
Subject: [PATCH] feat: accept date-only watched_at in review form

---
 .../template-askama/templates/new_review.html       |  2 +-
 crates/presentation/src/dtos.rs                     | 13 ++++++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/crates/adapters/template-askama/templates/new_review.html b/crates/adapters/template-askama/templates/new_review.html
index 912f2b7..f5ddbd0 100644
--- a/crates/adapters/template-askama/templates/new_review.html
+++ b/crates/adapters/template-askama/templates/new_review.html
@@ -29,7 +29,7 @@
   </label>
   <label>
     Watched<br>
-    <input type="datetime-local" name="watched_at" required>
+    <input type="date" name="watched_at" required>
   </label>
   <label>
     Comment<br>
diff --git a/crates/presentation/src/dtos.rs b/crates/presentation/src/dtos.rs
index 3b92823..94b1d7b 100644
--- a/crates/presentation/src/dtos.rs
+++ b/crates/presentation/src/dtos.rs
@@ -150,10 +150,14 @@ impl TryFrom<LogReviewForm> for LogReviewData {
     fn try_from(form: LogReviewForm) -> Result<Self, Self::Error> {
         let watched_at = NaiveDateTime::parse_from_str(&form.watched_at, "%Y-%m-%dT%H:%M:%S")
             .or_else(|_| NaiveDateTime::parse_from_str(&form.watched_at, "%Y-%m-%dT%H:%M"))
+            .or_else(|_| {
+                chrono::NaiveDate::parse_from_str(&form.watched_at, "%Y-%m-%d")
+                    .map(|d| d.and_hms_opt(0, 0, 0).expect("midnight always valid"))
+            })
             .map_err(|_| ParseReviewError {
                 field: "watched_at",
                 message: format!(
-                    "invalid date '{}'; expected YYYY-MM-DDTHH:MM[:SS]",
+                    "invalid date '{}'; expected YYYY-MM-DD or YYYY-MM-DDTHH:MM[:SS]",
                     form.watched_at
                 ),
             })?;
@@ -371,4 +375,11 @@ mod tests {
         let req: LoginRequest = serde_json::from_str(json).unwrap();
         assert_eq!(req.email, "a@b.com");
     }
+
+    #[test]
+    fn form_accepts_date_only() {
+        let data = LogReviewData::try_from(make_form("2024-03-15")).unwrap();
+        assert_eq!(data.watched_at.format("%H:%M:%S").to_string(), "00:00:00");
+        assert_eq!(data.watched_at.format("%Y-%m-%d").to_string(), "2024-03-15");
+    }
 }