From 6706f10864334a30a9e630a8e97e22b3e4816e79 Mon Sep 17 00:00:00 2001
From: Jonty Wareing <jonty@jonty.co.uk>
Date: Mon, 20 May 2024 23:58:32 +0100
Subject: [PATCH] Allow filtering of types in sense check

---
 apps/cfp_review/sense_check.py        | 8 ++++++--
 templates/cfp_review/sense_check.html | 9 +++++++++
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/apps/cfp_review/sense_check.py b/apps/cfp_review/sense_check.py
index 9caabb6e6..545564cde 100644
--- a/apps/cfp_review/sense_check.py
+++ b/apps/cfp_review/sense_check.py
@@ -1,6 +1,6 @@
 from datetime import timedelta
 
-from flask import render_template
+from flask import render_template, request
 
 from models import event_start, event_end
 from models.cfp import Proposal
@@ -97,9 +97,13 @@ def _check_timing(t, note, reason_key):
 @cfp_review.route("/sense_check")
 @review_required
 def sense_check():
+    types_to_show = request.args.getlist("type")
+    if not types_to_show:
+        types_to_show = ["talk", "workshop", "youthworkshop", "performance"]
+
     accepted_proposals = (
         Proposal.query_accepted(include_user_scheduled=False)
-        .filter(Proposal.type.in_(["talk", "workshop", "youthworkshop", "performance"]))
+        .filter(Proposal.type.in_(types_to_show))
         .order_by(Proposal.type)
         .all()
     )
diff --git a/templates/cfp_review/sense_check.html b/templates/cfp_review/sense_check.html
index 6142b0975..804861820 100644
--- a/templates/cfp_review/sense_check.html
+++ b/templates/cfp_review/sense_check.html
@@ -3,6 +3,15 @@
 {% block body %}
 <h2>Schedule Sense Check</h2>
 
+<p>
+  Show:
+  <a href="{{ url_for('cfp_review.sense_check') }}">All</a> /
+  <a href="{{ url_for('cfp_review.sense_check', type='talk') }}">Talk</a> /
+  <a href="{{ url_for('cfp_review.sense_check', type='workshop') }}">Workshop</a> /
+  <a href="{{ url_for('cfp_review.sense_check', type='youthworkshop') }}">Youth Workshop</a> /
+  <a href="{{ url_for('cfp_review.sense_check', type='performance') }}">Performance</a>
+</p>
+
 <p>Checked {{ proposals_count }} proposals, found {{ not_sensible_proposals|length }} suspect proposals.</p>
 
 {% if not_sensible_proposals|length > 0 %}