Merge branch 'main' into ms-hide-forecasted-grants

usdigitalresponse · Oct 17, 2024 · a27d07b · a27d07b
2 parents 6b1e358 + 2722baf
commit a27d07b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/server/src/arpa_reporter/services/validate-upload.js b/packages/server/src/arpa_reporter/services/validate-upload.js
@@ -326,9 +326,9 @@ async function validateRecord({ upload, record, typeRules: rules }) {
                 const lcItems = rule.listVals.map((val) => multiselect(val.toLowerCase()));
 
                 // for pick lists, the value must be one of possible values
-                if (rule.dataType === 'Pick List' && !lcItems.includes(value)) {
+                if (rule.dataType === 'Pick List' && !lcItems.includes(multiselect(value))) {
                     errors.push(new ValidationError(
-                        `Value for ${key} ('${value}') must be one of ${lcItems.length} options in the input template`,
+                        `Value for ${key} ('${value}') must be one of ${lcItems.length} valid options in the input template. (${JSON.stringify(lcItems)})`,
                         { col: rule.columnName, severity: 'err' },
                     ));
                 }