From dca7a68175ccf3f0a4e406ba203aa68dc6957727 Mon Sep 17 00:00:00 2001
From: Aleix <acasanov@redhat.com>
Date: Tue, 15 Nov 2022 09:28:20 +0100
Subject: [PATCH] Use editor component in the description field of a workshop
 (#694)

* Use editor component in the description field of a workshop

* fixup! Use editor component in the description field of a workshop

* Increase hash size to avoid collisions
---
 .../ui/src/app/Catalog/CatalogItemForm.tsx    | 25 +++++++++++++------
 catalog/ui/src/app/Services/service-utils.ts  |  2 +-
 .../app/Workshops/WorkshopsItemDetails.tsx    |  4 +--
 .../app/Workshops/WorkshopsItemServices.tsx   |  2 +-
 .../ui/src/app/components/Editor/Editor.tsx   | 18 ++++++++++---
 .../app/components/Editor/EditorViewer.tsx    | 17 +++++++++++--
 catalog/ui/webpack.common.js                  |  4 +--
 catalog/ui/webpack.prod.js                    |  4 +--
 8 files changed, 55 insertions(+), 21 deletions(-)

diff --git a/catalog/ui/src/app/Catalog/CatalogItemForm.tsx b/catalog/ui/src/app/Catalog/CatalogItemForm.tsx
index c682ef867..2ee96bdeb 100644
--- a/catalog/ui/src/app/Catalog/CatalogItemForm.tsx
+++ b/catalog/ui/src/app/Catalog/CatalogItemForm.tsx
@@ -1,6 +1,7 @@
 import React, { useEffect, useMemo, useReducer, useState } from 'react';
 import { useNavigate, useParams } from 'react-router-dom';
 import parseDuration from 'parse-duration';
+import { EditorState } from 'lexical/LexicalEditorState';
 import {
   ActionList,
   ActionListItem,
@@ -19,7 +20,6 @@ import {
   SelectOption,
   SelectVariant,
   Switch,
-  TextArea,
   TextInput,
   Title,
   Tooltip,
@@ -43,6 +43,7 @@ import { CatalogItem } from '@app/types';
 import { ErrorBoundary } from 'react-error-boundary';
 import { displayName, isLabDeveloper, randomString } from '@app/util';
 import DateTimePicker from '@app/components/DateTimePicker';
+import Editor from '@app/components/Editor/Editor';
 import useSession from '@app/utils/useSession';
 import useDebounce from '@app/utils/useDebounce';
 import PatientNumberInput from '@app/components/PatientNumberInput';
@@ -418,16 +419,24 @@ const CatalogItemFormData: React.FC<{ namespace: string; catalogItemName: string
             </FormGroup>
             <FormGroup fieldId="workshopDescription" label="Description">
               <div className="catalog-item-form__group-control--single">
-                <TextArea
-                  onChange={(v) =>
-                    dispatchFormState({ type: 'workshop', workshop: { ...formState.workshop, description: v } })
-                  }
-                  value={formState.workshop.description}
+                <Editor
+                  onChange={(state: EditorState) => {
+                    const editorState = state.toJSON();
+                    dispatchFormState({
+                      type: 'workshop',
+                      workshop: { ...formState.workshop, description: JSON.stringify(editorState) },
+                    });
+                  }}
+                  placeholder="Add description"
                   aria-label="Description"
+                  defaultValue={formState.workshop.description}
                 />
-                <Tooltip position="right" content={<p>Description shown in the workshop user interface.</p>}>
+                <Tooltip
+                  position="right"
+                  content={<p>Description text visible after the user accesses the Workshop.</p>}
+                >
                   <OutlinedQuestionCircleIcon
-                    aria-label="Description shown in the workshop user interface"
+                    aria-label="Description text visible after the user accesses the Workshop."
                     className="tooltip-icon-only"
                   />
                 </Tooltip>
diff --git a/catalog/ui/src/app/Services/service-utils.ts b/catalog/ui/src/app/Services/service-utils.ts
index 55abb18c4..636579352 100644
--- a/catalog/ui/src/app/Services/service-utils.ts
+++ b/catalog/ui/src/app/Services/service-utils.ts
@@ -14,7 +14,7 @@ export function getAutoTimes(resourceClaim: ResourceClaim): { startTime: number;
   function getSpecResourceByName(name: string): ResourceClaimSpecResource {
     return specResources.find((s) => s.name === name);
   }
-  if (!resources) return null;
+  if (!resources) return { startTime: null, stopTime: null };
   // The start time for a multi-component service is the earliest start time of all components and the stop time is the latest stop time
   const stopTimes = [];
   const startTimes = [];
diff --git a/catalog/ui/src/app/Workshops/WorkshopsItemDetails.tsx b/catalog/ui/src/app/Workshops/WorkshopsItemDetails.tsx
index f884c0ea6..7a1df347c 100644
--- a/catalog/ui/src/app/Workshops/WorkshopsItemDetails.tsx
+++ b/catalog/ui/src/app/Workshops/WorkshopsItemDetails.tsx
@@ -110,9 +110,9 @@ const WorkshopsItemDetails: React.FC<{
       <DescriptionListGroup>
         <DescriptionListTerm>
           Description{' '}
-          <Tooltip position="right" content={<p>Custom details visible after the user accesses the Workshop.</p>}>
+          <Tooltip position="right" content={<p>Description text visible after the user accesses the Workshop.</p>}>
             <OutlinedQuestionCircleIcon
-              aria-label="Custom details visible after the user accesses the Workshop."
+              aria-label="Description text visible after the user accesses the Workshop."
               className="tooltip-icon-only"
             />
           </Tooltip>
diff --git a/catalog/ui/src/app/Workshops/WorkshopsItemServices.tsx b/catalog/ui/src/app/Workshops/WorkshopsItemServices.tsx
index ca75d08cc..d6d247c08 100644
--- a/catalog/ui/src/app/Workshops/WorkshopsItemServices.tsx
+++ b/catalog/ui/src/app/Workshops/WorkshopsItemServices.tsx
@@ -52,7 +52,7 @@ const WorkshopsItemServices: React.FC<{
       <SelectableTable
         key="table"
         columns={['Name', 'GUID', 'Status', 'Created', 'Actions']}
-        onSelectAll={(isSelected) => {
+        onSelectAll={(isSelected: boolean) => {
           if (isSelected) {
             setSelectedUids(resourceClaims.map((resourceClaim) => resourceClaim.metadata.uid));
           } else {
diff --git a/catalog/ui/src/app/components/Editor/Editor.tsx b/catalog/ui/src/app/components/Editor/Editor.tsx
index ae3280058..7f4ce7344 100644
--- a/catalog/ui/src/app/components/Editor/Editor.tsx
+++ b/catalog/ui/src/app/components/Editor/Editor.tsx
@@ -1,6 +1,6 @@
 import React from 'react';
-import { EditorState, LexicalEditor } from 'lexical';
-import { LexicalComposer } from '@lexical/react/LexicalComposer';
+import { $createParagraphNode, $createTextNode, $getRoot, EditorState, LexicalEditor } from 'lexical';
+import { InitialEditorStateType, LexicalComposer } from '@lexical/react/LexicalComposer';
 import { RichTextPlugin } from '@lexical/react/LexicalRichTextPlugin';
 import { ContentEditable } from '@lexical/react/LexicalContentEditable';
 import { OnChangePlugin } from '@lexical/react/LexicalOnChangePlugin';
@@ -26,12 +26,24 @@ const Editor: React.FC<{
   placeholder: string;
   defaultValue?: string;
 }> = ({ onChange, placeholder, defaultValue }) => {
+  let _defaultValue: InitialEditorStateType = defaultValue;
+  try {
+    JSON.parse(defaultValue);
+  } catch {
+    _defaultValue = () => {
+      const root = $getRoot();
+      root.clear();
+      const p = $createParagraphNode();
+      p.append($createTextNode(defaultValue));
+      root.append(p);
+    };
+  }
   return (
     <div className="editor">
       <LexicalComposer
         initialConfig={{
           namespace: 'editor',
-          editorState: defaultValue,
+          editorState: _defaultValue,
           theme: Theme,
           nodes: [
             HeadingNode,
diff --git a/catalog/ui/src/app/components/Editor/EditorViewer.tsx b/catalog/ui/src/app/components/Editor/EditorViewer.tsx
index 656d4eea8..8475da093 100644
--- a/catalog/ui/src/app/components/Editor/EditorViewer.tsx
+++ b/catalog/ui/src/app/components/Editor/EditorViewer.tsx
@@ -1,5 +1,6 @@
 import React from 'react';
-import { LexicalComposer } from '@lexical/react/LexicalComposer';
+import { $createParagraphNode, $createTextNode, $getRoot } from 'lexical';
+import { InitialEditorStateType, LexicalComposer } from '@lexical/react/LexicalComposer';
 import { RichTextPlugin } from '@lexical/react/LexicalRichTextPlugin';
 import { ContentEditable } from '@lexical/react/LexicalContentEditable';
 import LexicalErrorBoundary from '@lexical/react/LexicalErrorBoundary';
@@ -15,13 +16,25 @@ import './editor.css';
 const EditorViewer: React.FC<{
   value?: string;
 }> = ({ value }) => {
+  let _defaultValue: InitialEditorStateType = value;
+  try {
+    JSON.parse(value);
+  } catch {
+    _defaultValue = () => {
+      const root = $getRoot();
+      root.clear();
+      const p = $createParagraphNode();
+      p.append($createTextNode(value));
+      root.append(p);
+    };
+  }
   return (
     <div className="editor editor-viewer">
       <LexicalComposer
         initialConfig={{
           editable: false,
           namespace: 'editor',
-          editorState: value,
+          editorState: _defaultValue,
           theme: Theme,
           nodes: [
             HeadingNode,
diff --git a/catalog/ui/webpack.common.js b/catalog/ui/webpack.common.js
index b108e8550..61edb6d0f 100644
--- a/catalog/ui/webpack.common.js
+++ b/catalog/ui/webpack.common.js
@@ -78,8 +78,8 @@ module.exports = () => {
     },
     output: {
       path: path.resolve(__dirname, 'dist'),
-      filename: '[name].[contenthash:4].bundle.js',
-      chunkFilename: '[name].[contenthash:4].bundle.js',
+      filename: '[name].[contenthash:8].bundle.js',
+      chunkFilename: '[name].[contenthash:8].bundle.js',
       publicPath: ASSET_PATH,
     },
     plugins: [
diff --git a/catalog/ui/webpack.prod.js b/catalog/ui/webpack.prod.js
index 1d7bf6a32..eda3d52ea 100644
--- a/catalog/ui/webpack.prod.js
+++ b/catalog/ui/webpack.prod.js
@@ -13,8 +13,8 @@ module.exports = merge(common('production'), {
   },
   plugins: [
     new MiniCssExtractPlugin({
-      filename: '[name].[contenthash:4].css',
-      chunkFilename: '[name].[contenthash:4].bundle.css',
+      filename: '[name].[contenthash:8].css',
+      chunkFilename: '[name].[contenthash:8].bundle.css',
       ignoreOrder: true,
     }),
   ],