fix type errors - make showBoundaryNames default to false

commonknowledge · Dec 18, 2024 · cbaac73 · cbaac73
1 parent 1368257
commit cbaac73
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/nextjs/src/app/reports/[id]/(components)/_ReportConfigLegacyControls.tsx b/nextjs/src/app/reports/[id]/(components)/_ReportConfigLegacyControls.tsx
@@ -82,10 +82,10 @@ const ReportConfigLegacyControls: React.FC = () => {
         </div>
         <div className="text-labelLg text-meepGray-200 flex items-center gap-2">
           <Switch
-            checked={showBoundaryNames}
-            onCheckedChange={(showBoundaryNames: boolean) => {
+            checked={showBoundaryNames ?? false}
+            onCheckedChange={(checked: boolean) => {
               updateReport({
-                displayOptions: { display: { showBoundaryNames } },
+                displayOptions: { display: { showBoundaryNames: checked } },
               })
             }}
           />

diff --git a/nextjs/src/app/reports/[id]/reportContext.ts b/nextjs/src/app/reports/[id]/reportContext.ts
@@ -33,7 +33,7 @@ export interface ReportConfig {
     showLastElectionData?: boolean
     showPostcodeLabels?: boolean
     boundaryOutlines?: AnalyticalAreaType[]
-    showBoundaryNames?: false
+    showBoundaryNames?: boolean
   }
 }
 
@@ -49,6 +49,7 @@ export const defaultReportConfig: ReportConfig = {
     showMPs: false,
     showLastElectionData: false,
     boundaryOutlines: [AnalyticalAreaType.ParliamentaryConstituency_2024],
+    showBoundaryNames: false,
   },
 }