Merge pull request #47 from saludangelito/cli

Added a checker if optionality required in addition to map type
run-crank · Feb 11, 2020 · 1865217 · 1865217
2 parents 1103794 + 19ca8af
commit 1865217
Showing 1 changed file with 25 additions and 5 deletions.
diff --git a/src/step-aware-command.ts b/src/step-aware-command.ts
@@ -270,11 +270,31 @@ export default abstract class extends RegistryAwareCommand {
       })
       expectedFields.forEach(field => {
         if (field.type === FieldDefinition.Type.ANYNONSCALAR || field.type === FieldDefinition.Type.MAP) {
-          prompts.next({
-            name: `:internal:confirm:${field.key}`,
-            message: `Add ${field.key} fields?`,
-            type: 'confirm'
-          })
+          if (field.optionality === FieldDefinition.Optionality.OPTIONAL) {
+            prompts.next({
+              name: `:internal:confirm:${field.key}`,
+              message: `Add ${field.key} fields?`,
+              type: 'confirm'
+            })
+          } else {
+            prompts.next({
+              name: `nonscalar.${field.key}.key`,
+              message: `${field.key} object key`,
+              type: 'input',
+              default: field.optionality === FieldDefinition.Optionality.OPTIONAL ? optionalMsg : null,
+            })
+            prompts.next({
+              name: `nonscalar.${field.key}.value`,
+              message: `${field.key} object value`,
+              type: 'input',
+              default: field.optionality === FieldDefinition.Optionality.OPTIONAL ? optionalMsg : null,
+            })
+            prompts.next({
+              name: ':internal:confirm:',
+              message: `Add another ${field.key} field?`,
+              type: 'confirm'
+            })
+          }
           hasObjectNeed = true
         } else {
           prompts.next({