Fixing issue #3199 - don't allow submitting automation creation modal without name, adding error message.

2021-11-03 11:55:01 +00:00 · 2021-11-03 11:55:01 +00:00 · 7b73a8dbe7
parent 484a38c9f5
commit 7b73a8dbe7
1 changed files with 10 additions and 2 deletions
--- a/packages/builder/src/components/automation/AutomationPanel/CreateAutomationModal.svelte
+++ b/packages/builder/src/components/automation/AutomationPanel/CreateAutomationModal.svelte
@ -8,10 +8,13 @@

  let name
  let selectedTrigger
+  let nameTouched = false
  let triggerVal
  export let webhookModal

  $: instanceId = $database._id
+  $: nameError =
+    nameTouched && !name ? "Please specify a name for the automation." : null

  async function createAutomation() {
    await automationStore.actions.create({
@ -51,13 +54,18 @@
  confirmText="Save"
  size="M"
  onConfirm={createAutomation}
-  disabled={!selectedTrigger}
+  disabled={!selectedTrigger || !name}
 >
  <Body size="XS"
    >Please name your automation, then select a trigger. Every automation must
    start with a trigger.
  </Body>
-  <Input bind:value={name} label="Name" />
+  <Input
+    bind:value={name}
+    on:change={() => (nameTouched = true)}
+    bind:error={nameError}
+    label="Name"
+  />

  <Layout noPadding>
    <Body size="S">Triggers</Body>