Add ‘Any’ option to school type filter

paulrobertlloyd · paulrobertlloyd · commit a7b818206b7b · 2026-04-14T16:43:37.000+01:00
diff --git a/app/controllers/school.js b/app/controllers/school.js
@@ -60,7 +60,7 @@ export const schoolController = {
     }
 
     // Filter by phase
-    if (phase) {
+    if (phase && phase !== 'none') {
       results = results.filter((school) => school.phase === phase)
     }
 
diff --git a/app/views/school/list.njk b/app/views/school/list.njk
@@ -36,7 +36,7 @@
             }
           },
           small: true,
-          items: enumItems(SchoolPhase),
+          items: radioFilterItems(SchoolPhase, data.phase),
           decorate: "phase"
         }) }}
 

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ export const schoolController = {`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`// Filter by phase`
`63`		`- if (phase) {`
	`63`	`+ if (phase && phase !== 'none') {`
`64`	`64`	`results = results.filter((school) => school.phase === phase)`
`65`	`65`	`}`
`66`	`66`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@`
`36`	`36`	`}`
`37`	`37`	`},`
`38`	`38`	`small: true,`
`39`		`- items: enumItems(SchoolPhase),`
	`39`	`+ items: radioFilterItems(SchoolPhase, data.phase),`
`40`	`40`	`decorate: "phase"`
`41`	`41`	`}) }}`
`42`	`42`