addressing review comments

appsmithorg · Sep 5, 2024 · f53f783 · f53f783
1 parent 247d01f
commit f53f783
Show file tree

Hide file tree

Showing 7 changed files with 15 additions and 93 deletions.
diff --git a/app/server/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/imageCaptioning.json b/app/server/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/imageCaptioning.json
@@ -39,17 +39,8 @@
           "configProperty": "actionConfiguration.formData.imageCaption.input.data",
           "controlType": "QUERY_DYNAMIC_INPUT_TEXT",
           "placeholderText": "Write some text or use {{ }} to reference a dynamic text value",
-          "isRequired": true,
-          "customStyles": {
-            "width": "700px"
-          }
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "IMAGE-CAPTION-Z3",
-      "children": [
+          "isRequired": true
+        },
         {
           "label": "Additional Instructions",
           "Description": "Provide additional instructions to tweak the caption",
@@ -58,10 +49,7 @@
           "controlType": "QUERY_DYNAMIC_TEXT",
           "placeholderText": "Write some text or use {{ }} to reference a dynamic text value",
           "isRequired": false,
-          "initialValue": "",
-          "customStyles": {
-            "width": "700px"
-          }
+          "initialValue": ""
         }
       ]
     }

diff --git a/...rver/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/imageClassification.json b/...rver/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/imageClassification.json
@@ -49,13 +49,7 @@
           "placeholderText": "Write some text or use {{ }} to reference a dynamic text value",
           "controlType": "QUERY_DYNAMIC_INPUT_TEXT",
           "isRequired": true
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "IMAGE-CLASSIFY-Z3",
-      "children": [
+        },
         {
           "label": "Labels",
           "Description": "Provide labels as comma-separated string input for classification",
@@ -64,13 +58,7 @@
           "controlType": "QUERY_DYNAMIC_INPUT_TEXT",
           "placeholderText": "Write a list of comma separated text values or use {{ }} to reference a dynamic value",
           "isRequired": true
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "IMAGE-CLASSIFY-Z4",
-      "children": [
+        },
         {
           "label": "Additional Instructions",
           "Description": "Provide additional instructions to tweak the classification",

diff --git a/...er/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/imageEntityExtraction.json b/...er/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/imageEntityExtraction.json
@@ -48,35 +48,17 @@
           "configProperty": "actionConfiguration.formData.imageEntity.input.data",
           "placeholderText": "Write some text or use {{ }} to reference a dynamic text value",
           "controlType": "QUERY_DYNAMIC_INPUT_TEXT",
-          "isRequired": true,
-          "customStyles": {
-            "width": "700px"
-          }
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "IMAGE-ENTITY-EXTRACT-Z3",
-      "children": [
+          "isRequired": true
+        },
         {
           "label": "Entities",
           "Description": "Provide a comma separated list of entities to extract from the Input",
           "subtitle": "Provide a comma separated list of entities to extract from the Input",
           "configProperty": "actionConfiguration.formData.imageEntity.labels.data",
           "controlType": "QUERY_DYNAMIC_INPUT_TEXT",
           "placeholderText": "Write a list of comma separated text values or use {{ }} to reference a dynamic value",
-          "isRequired": true,
-          "customStyles": {
-            "width": "700px"
-          }
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "IMAGE-ENTITY-EXTRACT-Z4",
-      "children": [
+          "isRequired": true
+        },
         {
           "label": "Additional Instructions",
           "Description": "Provide additional instructions for the AI to tweak the entity extraction",

diff --git a/...erver/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/textClassification.json b/...erver/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/textClassification.json
@@ -57,13 +57,7 @@
           "configProperty": "actionConfiguration.formData.textClassify.input.data",
           "controlType": "QUERY_DYNAMIC_TEXT",
           "isRequired": true
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "TEXT-CLASSIFY-Z3",
-      "children": [
+        },
         {
           "label": "Labels",
           "Description": "Provide a comma separated list of labels to classify the Input on",
@@ -72,13 +66,7 @@
           "placeholderText": "Write a list of comma separated text values or use {{ }} to reference a dynamic value",
           "controlType": "QUERY_DYNAMIC_INPUT_TEXT",
           "isRequired": true
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "TEXT-CLASSIFY-Z4",
-      "children": [
+        },
         {
           "label": "Additional Instructions",
           "Description": "Provide additional instructions for the AI to tweak the text classification",

diff --git a/...ver/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/textEntityExtraction.json b/...ver/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/textEntityExtraction.json
@@ -49,13 +49,7 @@
           "configProperty": "actionConfiguration.formData.textEntity.input.data",
           "controlType": "QUERY_DYNAMIC_TEXT",
           "isRequired": true
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "TEXT-ENTITY-EXTRACT-Z3",
-      "children": [
+        },
         {
           "label": "Entities",
           "Description": "Provide a comma separated list of entities to extract from the Input",
@@ -64,13 +58,7 @@
           "configProperty": "actionConfiguration.formData.textEntity.labels.data",
           "controlType": "QUERY_DYNAMIC_INPUT_TEXT",
           "isRequired": true
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "TEXT-ENTITY-EXTRACT-Z4",
-      "children": [
+        },
         {
           "label": "Additional Instructions",
           "Description": "Provide additional instructions for the AI to tweak the entity extraction",

diff --git a/...server/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/textSummarization.json b/...server/appsmith-plugins/appsmithAiPlugin/src/main/resources/editor/textSummarization.json
@@ -54,13 +54,7 @@
           "placeholderText": "Write some text or use {{ }} to reference a dynamic text value",
           "controlType": "QUERY_DYNAMIC_TEXT",
           "isRequired": true
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "TEXT-SUMMARY-Z3",
-      "children": [
+        },
         {
           "label": "Additional Instructions",
           "Description": "Provide additional instructions for the AI to tweak the summarization",

diff --git a/app/server/appsmith-plugins/smtpPlugin/src/main/resources/editor/body.json b/app/server/appsmith-plugins/smtpPlugin/src/main/resources/editor/body.json
@@ -88,13 +88,7 @@
           "controlType": "QUERY_DYNAMIC_TEXT",
           "evaluationSubstitutionType": "TEMPLATE",
           "placeholderText": "Incredible body text"
-        }
-      ]
-    },
-    {
-      "controlType": "SINGLE_COLUMN_ZONE",
-      "identifier": "BODY-Z6",
-      "children": [
+        },
         {
           "label": "Attachment(s)",
           "configProperty": "actionConfiguration.formData.send.attachments",