From 564645dbe8fafcfb2165098f444fa6b4f7e544f1 Mon Sep 17 00:00:00 2001
From: Sachin Chaurasiya <sachinchaurasiyachotey87@gmail.com>
Date: Wed, 4 Sep 2024 19:00:09 +0530
Subject: [PATCH] address comment

---
 .../Extensions/image/ImageComponent.tsx       | 77 +++----------------
 .../ui/src/locale/languages/de-de.json        |  2 +-
 .../ui/src/locale/languages/en-us.json        |  2 +-
 .../ui/src/locale/languages/es-es.json        |  2 +-
 .../ui/src/locale/languages/fr-fr.json        |  2 +-
 .../ui/src/locale/languages/he-he.json        |  2 +-
 .../ui/src/locale/languages/ja-jp.json        |  2 +-
 .../ui/src/locale/languages/nl-nl.json        |  2 +-
 .../ui/src/locale/languages/pt-br.json        |  2 +-
 .../ui/src/locale/languages/ru-ru.json        |  2 +-
 .../ui/src/locale/languages/zh-cn.json        |  2 +-
 11 files changed, 22 insertions(+), 75 deletions(-)

diff --git a/openmetadata-ui/src/main/resources/ui/src/components/BlockEditor/Extensions/image/ImageComponent.tsx b/openmetadata-ui/src/main/resources/ui/src/components/BlockEditor/Extensions/image/ImageComponent.tsx
index 958e06702c04..6321050a7642 100644
--- a/openmetadata-ui/src/main/resources/ui/src/components/BlockEditor/Extensions/image/ImageComponent.tsx
+++ b/openmetadata-ui/src/main/resources/ui/src/components/BlockEditor/Extensions/image/ImageComponent.tsx
@@ -23,10 +23,7 @@ import {
   Space,
   Tabs,
   Typography,
-  Upload,
-  UploadProps,
 } from 'antd';
-import { UploadChangeParam } from 'antd/lib/upload';
 import classNames from 'classnames';
 import { isEmpty } from 'lodash';
 import React, { FC, useState } from 'react';
@@ -48,36 +45,10 @@ const PopoverContent: FC<PopoverContentProps> = ({
   updateAttributes,
   onPopupVisibleChange,
   onUploadingChange,
-  isUploading,
-  deleteNode,
-  isValidSource,
   src,
+  deleteNode,
 }) => {
   const { t } = useTranslation();
-  const handleFileUploadChange = async (info: UploadChangeParam) => {
-    try {
-      const srcUrl = info.file.response?.result;
-      if (srcUrl) {
-        updateAttributes({ src: srcUrl, alt: info.file.fileName });
-        onPopupVisibleChange(false);
-      }
-    } catch (error) {
-      // handle error
-    } finally {
-      onUploadingChange(false);
-    }
-  };
-
-  const handleRequestUpload: UploadProps['customRequest'] = (options) => {
-    onUploadingChange(true);
-    const reader = new FileReader();
-    reader.readAsDataURL(options.file as Blob);
-    reader.addEventListener('load', (e) => {
-      setTimeout(() => {
-        options?.onSuccess?.(e.target);
-      }, 1000);
-    });
-  };
 
   const handleEmbedImage: FormProps['onFinish'] = (values) => {
     onPopupVisibleChange(false);
@@ -87,35 +58,6 @@ const PopoverContent: FC<PopoverContentProps> = ({
     onUploadingChange(false);
   };
 
-  const uploadElement = (
-    <Upload
-      accept="image/jpeg, image/png, image/webp"
-      className="om-node-image-upload"
-      customRequest={handleRequestUpload}
-      data-testid="image-upload-form"
-      multiple={false}
-      showUploadList={false}
-      onChange={handleFileUploadChange}>
-      <Space className="om-image-node-action">
-        <Button className="w-full" disabled={isUploading}>
-          {isValidSource ? t('label.update-image') : t('label.upload-image')}
-        </Button>
-        {isValidSource && (
-          <Button
-            className="w-full"
-            disabled={isUploading}
-            onClick={(e) => {
-              e.preventDefault();
-              e.stopPropagation();
-              deleteNode();
-            }}>
-            {t('label.delete')}
-          </Button>
-        )}
-      </Space>
-    </Upload>
-  );
-
   const embedLinkElement = (
     <Form
       data-testid="embed-link-form"
@@ -142,6 +84,16 @@ const PopoverContent: FC<PopoverContentProps> = ({
           </Form.Item>
         </Col>
         <Col className="om-image-node-embed-link-btn-col" span={24}>
+          <Space className="om-image-node-action">
+            <Button
+              onClick={(e) => {
+                e.preventDefault();
+                e.stopPropagation();
+                deleteNode();
+              }}>
+              {t('label.delete')}
+            </Button>
+          </Space>
           <Button htmlType="submit" type="primary">
             {t('label.embed-image')}
           </Button>
@@ -152,13 +104,8 @@ const PopoverContent: FC<PopoverContentProps> = ({
 
   return (
     <Tabs
-      defaultActiveKey="upload"
+      defaultActiveKey="embed"
       items={[
-        {
-          label: t('label.upload'),
-          key: 'upload',
-          children: uploadElement,
-        },
         {
           label: t('label.embed-link'),
           key: 'embed',
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/de-de.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/de-de.json
index d71865628b4b..e0cd50977770 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/de-de.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/de-de.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Select or enter HEX color code.",
     "hex-color-validation": "Input is not valid HEX code.",
     "hi-user-welcome-to": "Hallo {{user}}, Willkommen bei",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Sparen Sie Zeit und Aufwand, indem Sie eine CSV-Datei mit mehreren {{entity}} in einem Durchgang hochladen.",
     "in-this-database": "In dieser Datenbank",
     "include-assets-message": "Aktivieren Sie das Extrahieren von {{assets}} aus der Datenquelle.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/en-us.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/en-us.json
index 0fb23cff9563..1a2d8fc43339 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/en-us.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/en-us.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Select or enter HEX color code.",
     "hex-color-validation": "Input is not valid HEX code.",
     "hi-user-welcome-to": "Hi {{user}}, Welcome to",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Save time & effort by uploading a CSV file with several {{entity}} in one go.",
     "in-this-database": "In this Database",
     "include-assets-message": "Enable extracting {{assets}} from the data source.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/es-es.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/es-es.json
index 1ecc7921a040..9d5b37b68756 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/es-es.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/es-es.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Seleccione o ingrese el código de color HEX.",
     "hex-color-validation": "La entrada no es un código HEX válido.",
     "hi-user-welcome-to": "Hola {{user}}, Bienvenido a",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Ahorre tiempo y esfuerzo cargando un archivo CSV con varios {{entity}} de una vez.",
     "in-this-database": "En esta base de datos",
     "include-assets-message": "Configuración opcional para activar la ingestión de {{assets}}.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/fr-fr.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/fr-fr.json
index 2dabd8582ea9..b9548bca8632 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/fr-fr.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/fr-fr.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Sélectionnez un code couleur hexadécimal.",
     "hex-color-validation": "Code hexadécimal non valide.",
     "hi-user-welcome-to": "Bonjour {{user}}, Bienvenue sur",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Gagnez du temps et de l'effort en téléchargeant un fichier CSV contenant plusieurs {{entityPlural}} en une seule fois.",
     "in-this-database": "Dans cette base de données",
     "include-assets-message": "Activer l'extraction des {{assets}} de la source de données.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/he-he.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/he-he.json
index ff4ca429d8bc..d75e1177c3e8 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/he-he.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/he-he.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "בחר או הזן קוד צבע HEX.",
     "hex-color-validation": "הקלט אינו קוד HEX תקין.",
     "hi-user-welcome-to": "שלום {{user}}, ברוך הבא אל",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "חסוך זמן ומאמץ על ידי העלאת קובץ CSV עם מספר {{entity}} בפעם אחת.",
     "in-this-database": "במסד נתונים זה",
     "include-assets-message": "הפעל את החילוץ של {{assets}} ממקור הנתונים.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/ja-jp.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/ja-jp.json
index 2ed3b6fe29c4..c67250449833 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/ja-jp.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/ja-jp.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Select or enter HEX color code.",
     "hex-color-validation": "Input is not valid HEX code.",
     "hi-user-welcome-to": "Hi {{user}}, Welcome to",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Save time & effort by uploading a CSV file with several {{entity}} in one go.",
     "in-this-database": "In this Database",
     "include-assets-message": "データソースから{{assets}}の抽出を有効にする。",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/nl-nl.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/nl-nl.json
index a0335b1ead16..5d96bc4f763d 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/nl-nl.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/nl-nl.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Selecteer of voer de HEX-kleurcode in.",
     "hex-color-validation": "Invoer is geen geldige HEX-code.",
     "hi-user-welcome-to": "Hoi {{user}}, welkom bij",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Bespaar tijd en moeite door een CSV-bestand te uploaden met verschillende {{entity}} in één keer.",
     "in-this-database": "In deze database",
     "include-assets-message": "Schakel het extraheren van {{assets}} uit de databron in.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/pt-br.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/pt-br.json
index 26f86e68c979..971555755fed 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/pt-br.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/pt-br.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Selecione ou insira o código de cor HEX.",
     "hex-color-validation": "A entrada não é um código HEX válido.",
     "hi-user-welcome-to": "Olá {{user}}, Bem-vindo(a) ao",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Economize tempo e esforço fazendo upload de um arquivo CSV com vários {{entity}} de uma só vez.",
     "in-this-database": "Neste Banco de Dados",
     "include-assets-message": "Habilite a extração de {{assets}} da fonte de dados.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/ru-ru.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/ru-ru.json
index ff46dc108323..bed43c243b60 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/ru-ru.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/ru-ru.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "Select or enter HEX color code.",
     "hex-color-validation": "Input is not valid HEX code.",
     "hi-user-welcome-to": "Привет, {{user}}! Добро пожаловать в",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "Сэкономьте время и усилия, загрузив CSV-файл с несколькими {{entity}} за один раз.",
     "in-this-database": "В этой базе данных",
     "include-assets-message": "Включите извлечение {{assets}} из источника данных.",
diff --git a/openmetadata-ui/src/main/resources/ui/src/locale/languages/zh-cn.json b/openmetadata-ui/src/main/resources/ui/src/locale/languages/zh-cn.json
index eb7755178bee..a23235e9bb3f 100644
--- a/openmetadata-ui/src/main/resources/ui/src/locale/languages/zh-cn.json
+++ b/openmetadata-ui/src/main/resources/ui/src/locale/languages/zh-cn.json
@@ -1546,7 +1546,7 @@
     "hex-code-placeholder": "选择或输入 HEX 颜色代码",
     "hex-color-validation": "输入不是有效的 HEX 颜色代码",
     "hi-user-welcome-to": "Hi {{user}}, 欢迎来到",
-    "image-upload-error": "Image upload is not supported",
+    "image-upload-error": "Image upload is not supported. Please use Markdown syntax for images available via URL.",
     "import-entity-help": "通过上传 CSV 文件批量维护术语, 节约时间并提高效率。",
     "in-this-database": "在此数据库中",
     "include-assets-message": "启用从数据源提取{{assets}}",