From c1d9fd4021cc02052ce92c25215bcd061110b0ca Mon Sep 17 00:00:00 2001
From: Emi Griffith <me@emigriffith.dev>
Date: Sun, 19 Jul 2026 15:26:47 -0700
Subject: [PATCH] Hide footer email digest signup for now (#194)

* Hide footer email digest signup for now

Comment out the compact digest signup form in the site footer; restore
by removing the surrounding Jinja comment wrapper.

Claude-Session: https://claude.ai/code/session_01XXxmNFy9cZ6Gh8Y9thZn62

* Update digest test for hidden footer form

The footer signup form is now hidden; assert its absence site-wide.
The /digest endpoint stays live and its tests are unchanged.

Claude-Session: https://claude.ai/code/session_01XXxmNFy9cZ6Gh8Y9thZn62
---
 templates/base.html.j2 | 5 ++++-
 tests/test_digest.py   | 9 +++++----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/templates/base.html.j2 b/templates/base.html.j2
index 63665bf..c310b4a 100644
--- a/templates/base.html.j2
+++ b/templates/base.html.j2
@@ -65,7 +65,9 @@
 
   <footer class="site-footer">
     {# The compact digest signup is the site-wide footer pattern, not a
-       homepage-only surface — every page is a possible entry point. #}
+       homepage-only surface — every page is a possible entry point.
+       Hidden for now; restore by removing the surrounding {# … #} wrapper. #}
+    {#
     <form class="digest-form digest-compact" method="post" action="{{ base }}/digest"
           data-digest aria-labelledby="footer-digest-label">
       <label id="footer-digest-label" for="footer-digest-email">Your city's weather, graded — monthly</label>
@@ -77,6 +79,7 @@
       <p class="digest-note muted">No spam, no sharing your address, unsubscribe anytime.</p>
       <p class="digest-status" role="status" aria-live="polite" hidden></p>
     </form>
+    #}
     <nav aria-label="Footer">
       <a href="{{ base }}/climate">City climates</a>
       <a href="{{ base }}/glossary">Weather glossary</a>
diff --git a/tests/test_digest.py b/tests/test_digest.py
index 7b1d7ff..5de1022 100644
--- a/tests/test_digest.py
+++ b/tests/test_digest.py
@@ -112,9 +112,10 @@ def test_signup_records_event(client):
     assert sum(sum(days.values()) for days in after.values()) == before_n + 1
 
 
-def test_form_is_on_every_page(client):
-    """The digest form is the site-wide footer pattern, not homepage-only."""
+def test_form_is_hidden_for_now(client):
+    """The footer digest signup is temporarily hidden (commented out in the
+    base template). The /digest endpoint stays live; only the UI is pulled.
+    Restore the form and flip this back to asserting presence."""
     for path in ["/", "/about", "/climate", "/privacy", "/glossary"]:
         html = client.get(f"{B}{path}").text
-        assert "data-digest" in html, path
-        assert 'action="/thermograph/digest"' in html, path
+        assert "data-digest" not in html, path