From 930d9a53020a1e6522aeee03d6789cc218ea9083 Mon Sep 17 00:00:00 2001
From: Lucas Massemin <lucasmassemin@gmail.com>
Date: Thu, 5 Dec 2024 15:14:56 +0100
Subject: [PATCH] listing file extensions, moving version history up (#9148)

Co-authored-by: Lucas <lucas@dust.tt>
---
 .../assistant_builder/InstructionScreen.tsx   | 78 +++++++++----------
 .../data_source/DocumentUploadOrEditModal.tsx |  2 +-
 2 files changed, 40 insertions(+), 40 deletions(-)

diff --git a/front/components/assistant_builder/InstructionScreen.tsx b/front/components/assistant_builder/InstructionScreen.tsx
index d59c1f1baefb..32262b4f1499 100644
--- a/front/components/assistant_builder/InstructionScreen.tsx
+++ b/front/components/assistant_builder/InstructionScreen.tsx
@@ -282,6 +282,45 @@ export function InstructionScreen({
           </Page.P>
         </div>
         <div className="flex-grow" />
+        {configsWithUniqueInstructions &&
+          configsWithUniqueInstructions.length > 1 &&
+          currentConfig && (
+            <div className="mr-2 mt-2 self-end">
+              <PromptHistory
+                history={configsWithUniqueInstructions}
+                onConfigChange={(config) => {
+                  // Remember the instructions of the version we're leaving, if overriden
+                  if (
+                    currentConfig &&
+                    currentConfig.instructions !== builderState.instructions
+                  ) {
+                    setOverridenConfigInstructions((prev) => ({
+                      ...prev,
+                      [currentConfig.version]: builderState.instructions,
+                    }));
+                  }
+
+                  // Bring new version's instructions to the editor, fetch overriden instructions if any
+                  setCurrentConfig(config);
+                  editorService.resetContent(
+                    tipTapContentFromPlainText(
+                      overridenConfigInstructions[config.version] ||
+                        config.instructions ||
+                        ""
+                    )
+                  );
+                  setBuilderState((state) => ({
+                    ...state,
+                    instructions:
+                      overridenConfigInstructions[config.version] ||
+                      config.instructions ||
+                      "",
+                  }));
+                }}
+                currentConfig={currentConfig}
+              />
+            </div>
+          )}
         <div className="mt-2 self-end">
           <AdvancedSettings
             owner={owner}
@@ -299,45 +338,6 @@ export function InstructionScreen({
       </div>
       <div className="flex h-full flex-col gap-1">
         <div className="relative h-full min-h-[240px] grow gap-1 p-px">
-          {configsWithUniqueInstructions &&
-            configsWithUniqueInstructions.length > 1 &&
-            currentConfig && (
-              <div className="absolute right-2 top-2 z-10">
-                <PromptHistory
-                  history={configsWithUniqueInstructions}
-                  onConfigChange={(config) => {
-                    // Remember the instructions of the version we're leaving, if overriden
-                    if (
-                      currentConfig &&
-                      currentConfig.instructions !== builderState.instructions
-                    ) {
-                      setOverridenConfigInstructions((prev) => ({
-                        ...prev,
-                        [currentConfig.version]: builderState.instructions,
-                      }));
-                    }
-
-                    // Bring new version's instructions to the editor, fetch overriden instructions if any
-                    setCurrentConfig(config);
-                    editorService.resetContent(
-                      tipTapContentFromPlainText(
-                        overridenConfigInstructions[config.version] ||
-                          config.instructions ||
-                          ""
-                      )
-                    );
-                    setBuilderState((state) => ({
-                      ...state,
-                      instructions:
-                        overridenConfigInstructions[config.version] ||
-                        config.instructions ||
-                        "",
-                    }));
-                  }}
-                  currentConfig={currentConfig}
-                />
-              </div>
-            )}
           <EditorContent
             editor={editor}
             className="absolute bottom-0 left-0 right-0 top-0"
diff --git a/front/components/data_source/DocumentUploadOrEditModal.tsx b/front/components/data_source/DocumentUploadOrEditModal.tsx
index 03dc69fb213d..b3aa0ee40273 100644
--- a/front/components/data_source/DocumentUploadOrEditModal.tsx
+++ b/front/components/data_source/DocumentUploadOrEditModal.tsx
@@ -349,7 +349,7 @@ export const DocumentUploadOrEditModal = ({
               <div>
                 <Page.SectionHeader
                   title="Text content"
-                  description={`Copy paste content or upload a file (text or PDF). Up to ${
+                  description={`Copy paste content or upload a file (${supportedPlainTextExtensions.join(", ")}). \n Up to ${
                     plan.limits.dataSources.documents.sizeMb === -1
                       ? "2"
                       : plan.limits.dataSources.documents.sizeMb