From 41805bc3f22d3402a9b2661843d96300c90f8d70 Mon Sep 17 00:00:00 2001
From: Rory Doak <rory@opensystemslab.io>
Date: Mon, 24 Jun 2024 11:15:25 +0100
Subject: [PATCH] add validation to isPlanningDataCollected bool

---
 .../Settings/GeneralSettings/HomepagePlanningForm.tsx         | 4 +++-
 .../FlowEditor/components/Settings/GeneralSettings/index.tsx  | 2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/HomepagePlanningForm.tsx b/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/HomepagePlanningForm.tsx
index 5871c23728..51e806b6ac 100644
--- a/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/HomepagePlanningForm.tsx
+++ b/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/HomepagePlanningForm.tsx
@@ -56,7 +56,9 @@ export default function HomepagePlanningForm({ formikConfig }: FormProps) {
               Do you collect Planning data?
               <RadioGroup
                 name="isPlanningDataCollected"
-                defaultValue={"Yes"}
+                defaultValue={
+                  formik.values.isPlanningDataCollected === true ? "yes" : "no"
+                }
                 onChange={(event) => {
                   formik.setFieldValue(
                     "isPlanningDataCollected",
diff --git a/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/index.tsx b/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/index.tsx
index 9605ede78c..9d665aa4a0 100644
--- a/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/index.tsx
+++ b/editor.planx.uk/src/pages/FlowEditor/components/Settings/GeneralSettings/index.tsx
@@ -48,7 +48,7 @@ const GeneralSettings: React.FC = () => {
     helpPhone: "",
     helpOpeningHours: "",
     homepage: "",
-    isPlanningDataCollected: false,
+    isPlanningDataCollected: true,
     portalName: "",
     portalUrl: "",
   };