Update addon validation

src/commands/addons/create.js

@@ -72,10 +72,15 @@ class AddonsCreateCommand extends Command {
           console.log(
             `${chalk.redBright.underline.bold(`Error: Missing required configuration for "${addonName} add-on"`)}`
           )
-          console.log(`Please supply the configuration values as CLI flags`)
           console.log()
           render.missingValues(missingValues, manifest)
           console.log()
+          const msg = `netlify addons:create ${addonName}`
+          console.log(`Please supply the configuration values as CLI flags`)
+          console.log()
+          console.log(`Alternatively, you can run ${chalk.cyan(msg)} with no flags to walk through the setup steps`)
+          console.log()
+          return false
         }
 
         await createSiteAddon({
@@ -110,6 +115,13 @@ class AddonsCreateCommand extends Command {
       const userInput = await inquirer.prompt(prompts)
       // Merge user input with the flags specified
       configValues = updateConfigValues(manifest.config, rawFlags, userInput)
+      const missingRequiredValues = missingConfigValues(required, configValues)
+      if (missingRequiredValues && missingRequiredValues.length) {
+        missingRequiredValues.forEach((val) => {
+          console.log(`Missing required value "${val}". Please run the command again`)
+        })
+        return false
+      }
     }
 
     await createSiteAddon({

src/commands/deploy.js

@@ -69,10 +69,12 @@ class DeployCommand extends Command {
     }
 
     let functionsFolder
+    // Support "functions" and "Functions"
+    const funcConfig = get(config, 'build.functions') || get(config, 'build.Functions')
     if (flags['functions']) {
       functionsFolder = path.resolve(process.cwd(), flags['functions'])
-    } else if (get(config, 'build.functions')) {
-      functionsFolder = path.resolve(site.root, get(config, 'build.functions'))
+    } else if (funcConfig) {
+      functionsFolder = path.resolve(site.root, funcConfig)
     } else if (get(siteData, 'build_settings.functions_dir')) {
       functionsFolder = path.resolve(site.root, get(siteData, 'build_settings.functions_dir'))
     }

src/utils/addons/prompts.js

@@ -38,13 +38,16 @@ module.exports = function generatePrompts(settings) {
       return prompt
     }
 
-    const validateFunction = (setting.pattern) ? validate(setting.pattern) : noValidate
-
+    // For future use. Once UX is decided
+    // const defaultValidation = (setting.required) ? validateRequired : noValidate
+    const defaultValidation = noValidate
+    const validateFunction = (setting.pattern) ? validate(setting.pattern) : defaultValidation
+    const isRequiredText = (setting.required) ? ` (${chalk.yellow('required')})` : ''
     if (setting.type === 'string' || setting.type.match((/string/))) {
       prompt = {
         type: 'input',
         name: key,
-        message: `${chalk.white(key)} - ${setting.displayName}` || `Please enter value for ${key}`,
+        message: `${chalk.white(key)}${isRequiredText} - ${setting.displayName}` || `Please enter value for ${key}`,
         validate: validateFunction
       }
       // if value previously set show it
@@ -67,6 +70,14 @@ function noValidate() {
   return true
 }
 
+// Will use this soon
+function validateRequired(value) { // eslint-disable-line 
+  if (value) {
+    return true
+  }
+  return `Please enter a value this field is required`
+}
+
 function validate(pattern) {
   return function(value) {
     const regex = new RegExp(pattern)