Show collaboration details and add acknowledge action for non-PR requests

app.py

@@ -56,18 +56,33 @@ def history_rows():
 def load_submission(submission_id: str):
     """Populate the detail panel + editable fields for a selected submission."""
     if not submission_id:
-        return (gr.update(value=""), *[gr.update(visible=False) for _ in ALL_FIELD_KEYS], gr.update(visible=False))
+        return (
+            gr.update(value=""),
+            *[gr.update(visible=False) for _ in ALL_FIELD_KEYS],
+            gr.update(visible=False),
+            gr.update(visible=False),
+        )
 
     row = submissions.get_submission(submission_id)
     if row is None:
-        return (gr.update(value="Not found."), *[gr.update(visible=False) for _ in ALL_FIELD_KEYS], gr.update(visible=False))
+        return (
+            gr.update(value="Not found."),
+            *[gr.update(visible=False) for _ in ALL_FIELD_KEYS],
+            gr.update(visible=False),
+            gr.update(visible=False),
+        )
 
     detail_md = (
         f"**Type:** {row['type']}  \n"
         f"**Title/link:** {row['title'] or '_none given_'}  \n"
-        f"**Submitted:** {row['submitted_at']} via {row['source']}  \n\n"
-        f"{row['description']}"
+        f"**Submitted:** {row['submitted_at']} via {row['source']}  \n"
     )
+    if row["type"] == "collaboration":
+        detail_md += (
+            f"**Email:** {row.get('email') or '_none given_'}  \n"
+            f"**Institution:** {row.get('institution') or '_none given_'}  \n"
+        )
+    detail_md += f"\n{row['description']}"
 
     visible = FIELD_VISIBILITY.get(row["type"], set())
     guessed_slug = _guess_slug(row["title"])
@@ -88,7 +103,13 @@ def load_submission(submission_id: str):
             field_updates.append(gr.update(visible=False))
 
     show_pr_button = row["type"] in PR_TYPES
-    return (gr.update(value=detail_md), *field_updates, gr.update(visible=show_pr_button))
+    show_ack_button = row["type"] not in PR_TYPES
+    return (
+        gr.update(value=detail_md),
+        *field_updates,
+        gr.update(visible=show_pr_button),
+        gr.update(visible=show_ack_button),
+    )
 
 
 with gr.Blocks(title="Hugging Science — requests review") as demo:
@@ -121,6 +142,7 @@ with gr.Blocks(title="Hugging Science — requests review") as demo:
 
             with gr.Row():
                 approve_btn = gr.Button("Approve → open PR", variant="primary", visible=False)
+                ack_btn = gr.Button("Mark as reviewed", variant="primary", visible=False)
                 reject_reason = gr.Textbox(label="Reject reason (optional)", scale=2)
                 reject_btn = gr.Button("Reject", variant="stop")
 
@@ -132,7 +154,7 @@ with gr.Blocks(title="Hugging Science — requests review") as demo:
             refresh_btn.click(do_refresh, outputs=pending_dd)
             demo.load(do_refresh, outputs=pending_dd)
 
-            pending_dd.change(load_submission, inputs=pending_dd, outputs=[detail, *all_fields, approve_btn])
+            pending_dd.change(load_submission, inputs=pending_dd, outputs=[detail, *all_fields, approve_btn, ack_btn])
 
             def do_approve(submission_id, id_, name, title, slug, org_id, entry_type,
                             description, excerpt, link, date, featured, upvotes, tags):
@@ -161,6 +183,14 @@ with gr.Blocks(title="Hugging Science — requests review") as demo:
                 outputs=[result_md, pending_dd],
             )
 
+            def do_ack(submission_id):
+                if not submission_id:
+                    return "Nothing selected.", gr.update()
+                submissions.mark_acknowledged(submission_id)
+                return "Marked as reviewed.", gr.update(choices=pending_choices(), value=None)
+
+            ack_btn.click(do_ack, inputs=[pending_dd], outputs=[result_md, pending_dd])
+
             def do_reject(submission_id, reason):
                 if not submission_id:
                     return "Nothing selected.", gr.update()

submissions.py

@@ -71,6 +71,19 @@ def mark_approved(submission_id: str, pr_url: str) -> None:
         _write_all(rows)
 
 
+def mark_acknowledged(submission_id: str) -> None:
+    """For non-PR types (collaboration, challenge): close out a request
+    without opening a PR — e.g. once the reviewer has emailed the requester."""
+    with _lock:
+        rows = _read_all()
+        for row in rows:
+            if row["id"] == submission_id:
+                row["status"] = "acknowledged"
+                row["reviewed_at"] = _now()
+                break
+        _write_all(rows)
+
+
 def mark_rejected(submission_id: str, reason: str | None) -> None:
     with _lock:
         rows = _read_all()