fixing type for ParameterSchema

Kira-Pilot · Kira-Pilot · commit f8dcb1b2ec3c · 2022-06-15T18:43:24.000Z
resolves #2161
diff --git a/codersdk/parameters.go b/codersdk/parameters.go
@@ -72,7 +72,7 @@ type ParameterSchema struct {
 
 	// This is a special array of items provided if the validation condition
 	// explicitly states the value must be one of a set.
-	ValidationContains []string `json:"validation_contains"`
+	ValidationContains []string `json:"validation_contains,omitempty"`
 }
 
 // CreateParameterRequest is used to create a new parameter value for a scope.
diff --git a/site/src/api/typesGenerated.ts b/site/src/api/typesGenerated.ts
@@ -190,7 +190,7 @@ export interface ParameterSchema {
   readonly validation_condition: string
   readonly validation_type_system: string
   readonly validation_value_type: string
-  readonly validation_contains: string[]
+  readonly validation_contains?: string[]
 }
 
 // From codersdk/provisionerdaemons.go:33:6
diff --git a/site/src/components/ParameterInput/ParameterInput.tsx b/site/src/components/ParameterInput/ParameterInput.tsx
@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-unnecessary-condition */
 import FormControlLabel from "@material-ui/core/FormControlLabel"
 import Radio from "@material-ui/core/Radio"
 import RadioGroup from "@material-ui/core/RadioGroup"
@@ -38,16 +37,15 @@ const ParameterField: React.FC<ParameterInputProps> = ({ disabled, onChange, sch
           onChange(event.target.value)
         }}
       >
-        {schema.validation_contains &&
-          schema.validation_contains.map((item) => (
-            <FormControlLabel
-              disabled={disabled}
-              key={item}
-              value={item}
-              control={<Radio color="primary" size="small" disableRipple />}
-              label={item}
-            />
-          ))}
+        {schema.validation_contains.map((item) => (
+          <FormControlLabel
+            disabled={disabled}
+            key={item}
+            value={item}
+            control={<Radio color="primary" size="small" disableRipple />}
+            label={item}
+          />
+        ))}
       </RadioGroup>
     )
   }

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ type ParameterSchema struct {`
`72`	`72`
`73`	`73`	`// This is a special array of items provided if the validation condition`
`74`	`74`	`// explicitly states the value must be one of a set.`
`75`		- ValidationContains []string `json:"validation_contains"`
	`75`	+ ValidationContains []string `json:"validation_contains,omitempty"`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`// CreateParameterRequest is used to create a new parameter value for a scope.`
Original file line number	Diff line number	Diff line change
`@@ -190,7 +190,7 @@ export interface ParameterSchema {`
`190`	`190`	`readonly validation_condition: string`
`191`	`191`	`readonly validation_type_system: string`
`192`	`192`	`readonly validation_value_type: string`
`193`		`- readonly validation_contains: string[]`
	`193`	`+ readonly validation_contains?: string[]`
`194`	`194`	`}`
`195`	`195`
`196`	`196`	`// From codersdk/provisionerdaemons.go:33:6`