From 9910ce0ec4c4439a40a50db28ff49ea731c895fa Mon Sep 17 00:00:00 2001
From: Bendik Aagaard Lynghaug <bendik.lynghaug@gmail.com>
Date: Wed, 5 Aug 2026 14:23:38 +0200
Subject: [PATCH] Fold newsletter compose into Subscribers as a second feature,
 not its own alternative

Per feedback: sending a newsletter is naturally part of "Subscribers",
not a separate review task. Also fixes a real bug - the standalone
Compose newsletter alternative had no `action`, so AlternativeCard
never rendered a submit button/form for it at all (has_action gates
the <form>, unconditionally). action: /review reloads the dashboard
after sending.
---
 questions/review.yaml | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/questions/review.yaml b/questions/review.yaml
index 898c322..0aea7cf 100644
--- a/questions/review.yaml
+++ b/questions/review.yaml
@@ -33,21 +33,18 @@ alternatives:
 
   - name: Subscribers
     description: ""
+    action: /review
+    consequence: [Send]
     features:
       - name: ""
         description: ""
         resource:
           bucket: portal_subscribers
           requires_group: owners
-
-  - name: Compose newsletter
-    description: >
-      Sends to everyone in Subscribers - picked up by an automation
-      watching this question, not portal itself.
-    consequence: [Send]
-    features:
-      - name: ""
-        description: ""
+      - name: Compose newsletter
+        description: >
+          Sends to everyone above - picked up by an automation watching
+          this question, not portal itself.
         requirements:
           - name: subject
             label: Subject