🚸 (webhook) Show save in variables accordion only when needed

baptisteArno · Jun 16, 2023 · e54aab4 · e54aab4 · vercel · Jun 16, 2023
1 parent a64e621
commit e54aab4
Showing 1 changed file with 18 additions and 16 deletions.
diff --git a/...builder/src/features/blocks/integrations/webhook/components/WebhookAdvancedConfigForm.tsx b/...builder/src/features/blocks/integrations/webhook/components/WebhookAdvancedConfigForm.tsx
@@ -208,22 +208,24 @@ export const WebhookAdvancedConfigForm = ({
       {testResponse && (
         <CodeEditor isReadOnly lang="json" value={testResponse} />
       )}
-      <Accordion allowMultiple>
-        <AccordionItem>
-          <AccordionButton justifyContent="space-between">
-            Save in variables
-            <AccordionIcon />
-          </AccordionButton>
-          <AccordionPanel pt="4">
-            <TableList<ResponseVariableMapping>
-              initialItems={options.responseVariableMapping}
-              onItemsChange={updateResponseVariableMapping}
-              Item={ResponseMappingInputs}
-              addLabel="Add an entry"
-            />
-          </AccordionPanel>
-        </AccordionItem>
-      </Accordion>
+      {(testResponse || options.responseVariableMapping.length > 0) && (
+        <Accordion allowMultiple>
+          <AccordionItem>
+            <AccordionButton justifyContent="space-between">
+              Save in variables
+              <AccordionIcon />
+            </AccordionButton>
+            <AccordionPanel pt="4">
+              <TableList<ResponseVariableMapping>
+                initialItems={options.responseVariableMapping}
+                onItemsChange={updateResponseVariableMapping}
+                Item={ResponseMappingInputs}
+                addLabel="Add an entry"
+              />
+            </AccordionPanel>
+          </AccordionItem>
+        </Accordion>
+      )}
     </>
   )
 }