From 1aceaea3c2c4324800ecb563171482f01c4b7bf6 Mon Sep 17 00:00:00 2001
From: Bilal Meddah <bilal.meddah@epfl.ch>
Date: Tue, 20 Feb 2024 14:59:43 +0100
Subject: [PATCH 1/2] fix: omit the resource metadata from the update endpoint

---
 src/pages/StudiosPage/StudiosPage.tsx                  |  2 +-
 src/shared/styles/route-layout.less                    | 10 ++++++++++
 .../studioLegacy/containers/StudioContainer.tsx        |  8 ++++++--
 3 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/src/pages/StudiosPage/StudiosPage.tsx b/src/pages/StudiosPage/StudiosPage.tsx
index 1a453a694..6ba2db18a 100644
--- a/src/pages/StudiosPage/StudiosPage.tsx
+++ b/src/pages/StudiosPage/StudiosPage.tsx
@@ -124,7 +124,7 @@ const StudioItem = ({
               )}
             </h3>
           </Link>
-          <p>{description}</p>
+          <p className="description">{description}</p>
         </div>
         <div className="statistics studios-list-item">
           <div className="statistics_item">
diff --git a/src/shared/styles/route-layout.less b/src/shared/styles/route-layout.less
index f377ffc82..385bd8629 100644
--- a/src/shared/styles/route-layout.less
+++ b/src/shared/styles/route-layout.less
@@ -132,6 +132,7 @@
                   text-align: center;
                   vertical-align: middle;
                 }
+
                 .depreacted-tag {
                   font-family: 'Titillium Web';
                   font-style: normal;
@@ -143,6 +144,7 @@
                   margin: 0 3px;
                   vertical-align: middle;
                 }
+
                 .deletion-tag {
                   font-family: 'Titillium Web';
                   font-style: normal;
@@ -168,6 +170,14 @@
               }
             }
 
+            .description {
+              overflow: hidden;
+              display: -webkit-box;
+              -webkit-box-orient: vertical;
+              -webkit-line-clamp: 2;
+              line-clamp: 2;
+            }
+
             .statistics {
               display: grid;
               grid-template-columns: 1fr 1fr;
diff --git a/src/subapps/studioLegacy/containers/StudioContainer.tsx b/src/subapps/studioLegacy/containers/StudioContainer.tsx
index e0e4891c8..edc1ae343 100644
--- a/src/subapps/studioLegacy/containers/StudioContainer.tsx
+++ b/src/subapps/studioLegacy/containers/StudioContainer.tsx
@@ -2,7 +2,9 @@ import * as React from 'react';
 import { Resource } from '@bbp/nexus-sdk';
 import { useNexusContext, AccessControl } from '@bbp/react-nexus';
 import { Empty, message } from 'antd';
+import { omitBy } from 'lodash';
 import { useHistory } from 'react-router';
+
 import EditStudio from '../components/EditStudio';
 import StudioHeader from '../components/StudioHeader';
 import StudioReactContext from '../contexts/StudioContext';
@@ -114,7 +116,9 @@ const StudioContainer: React.FunctionComponent = () => {
         studioId,
         studioResource._rev,
         {
-          ...studioResource,
+          // remove the metadata from the payload, delta do full update
+          // and not accept the metadata fields to be in the payload
+          ...omitBy(studioResource, (_, key) => key.trim().startsWith('_')),
           label,
           description,
           plugins,
@@ -145,7 +149,7 @@ const StudioContainer: React.FunctionComponent = () => {
       onSave={updateStudio}
       onSaveImage={saveImage(nexus, orgLabel, projectLabel)}
       markdownViewer={MarkdownViewerContainer}
-    ></EditStudio>
+    />
   );
   return (
     <>

From 71ea84beea889a2518f4f7e2e49dd772e9c5ca06 Mon Sep 17 00:00:00 2001
From: Dinika Saxena <dinikasaxenas@gmail.com>
Date: Tue, 20 Feb 2024 17:28:48 +0100
Subject: [PATCH 2/2] Remove metadata when updating studios or workspaces

Signed-off-by: Dinika Saxena <dinikasaxenas@gmail.com>
---
 src/shared/containers/DataTableContainer.tsx             | 3 ++-
 src/subapps/studioLegacy/containers/StudioContainer.tsx  | 6 +++++-
 .../studioLegacy/containers/WorkspaceFormContainer.tsx   | 3 ++-
 .../studioLegacy/containers/WorkspaceMenuContainer.tsx   | 9 +++++----
 4 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/src/shared/containers/DataTableContainer.tsx b/src/shared/containers/DataTableContainer.tsx
index 779d139ac..69a85f2a4 100644
--- a/src/shared/containers/DataTableContainer.tsx
+++ b/src/shared/containers/DataTableContainer.tsx
@@ -37,6 +37,7 @@ import {
   DataPanelEvent,
 } from '../organisms/DataPanel/DataPanel';
 import { TResourceTableData } from '../molecules/MyDataTable/MyDataTable';
+import { resourceWithoutMetadata } from '../../subapps/studioLegacy/containers/StudioContainer';
 
 export type TableColumn = {
   '@type': string;
@@ -290,7 +291,7 @@ const DataTableContainer: React.FC<DataTableProps> = ({
         projectLabel,
         encodeURIComponent(data['@id']),
         latest._rev,
-        { ...latest, ...data }
+        { ...resourceWithoutMetadata(latest), ...resourceWithoutMetadata(data) }
       );
     }
     const resource = await nexus.Resource.create(orgLabel, projectLabel, data);
diff --git a/src/subapps/studioLegacy/containers/StudioContainer.tsx b/src/subapps/studioLegacy/containers/StudioContainer.tsx
index edc1ae343..f886c408c 100644
--- a/src/subapps/studioLegacy/containers/StudioContainer.tsx
+++ b/src/subapps/studioLegacy/containers/StudioContainer.tsx
@@ -16,6 +16,10 @@ import useNotification, {
   parseNexusError,
 } from '../../../shared/hooks/useNotification';
 
+export const resourceWithoutMetadata = (
+  studioResource: StudioResource | Resource
+) => omitBy(studioResource, (_, key) => key.trim().startsWith('_'));
+
 const resourcesWritePermissionsWrapper = (
   child: React.ReactNode,
   permissionPath: string
@@ -118,7 +122,7 @@ const StudioContainer: React.FunctionComponent = () => {
         {
           // remove the metadata from the payload, delta do full update
           // and not accept the metadata fields to be in the payload
-          ...omitBy(studioResource, (_, key) => key.trim().startsWith('_')),
+          ...resourceWithoutMetadata(studioResource),
           label,
           description,
           plugins,
diff --git a/src/subapps/studioLegacy/containers/WorkspaceFormContainer.tsx b/src/subapps/studioLegacy/containers/WorkspaceFormContainer.tsx
index 824c099e3..eb7c22fe3 100644
--- a/src/subapps/studioLegacy/containers/WorkspaceFormContainer.tsx
+++ b/src/subapps/studioLegacy/containers/WorkspaceFormContainer.tsx
@@ -19,6 +19,7 @@ import {
 } from 'antd';
 import TextArea from 'antd/lib/input/TextArea';
 import useNotification from '../../../shared/hooks/useNotification';
+import { resourceWithoutMetadata } from './StudioContainer';
 
 type NexusSparqlError = {
   reason: string;
@@ -123,7 +124,7 @@ const WorkspaceForm: React.FunctionComponent<WorkspaceFormProps> = ({
       projectLabel,
       encodeURIComponent(workspace['@id']),
       workspace['_rev'],
-      newWorkspace
+      resourceWithoutMetadata(newWorkspace)
     )
       .then(result => {
         if (onSuccess) {
diff --git a/src/subapps/studioLegacy/containers/WorkspaceMenuContainer.tsx b/src/subapps/studioLegacy/containers/WorkspaceMenuContainer.tsx
index 06f471955..489346073 100644
--- a/src/subapps/studioLegacy/containers/WorkspaceMenuContainer.tsx
+++ b/src/subapps/studioLegacy/containers/WorkspaceMenuContainer.tsx
@@ -11,7 +11,7 @@ import {
 import PromisePool from '@supercharge/promise-pool';
 import { ItemType } from 'antd/lib/menu/hooks/useItems';
 import { useQuery } from 'react-query';
-import { find, orderBy } from 'lodash';
+import { find, omitBy, orderBy } from 'lodash';
 import useNotification from '../../../shared/hooks/useNotification';
 import EditTableForm from '../../../shared/components/EditTableForm';
 import DashboardEditorContainer from './DashBoardEditor/DashboardEditorContainer';
@@ -30,6 +30,7 @@ import STUDIO_CONTEXT from '../components/StudioContext';
 import { createTableContext } from '../../../subapps/projects/utils/workFlowMetadataUtils';
 import { ErrorComponent } from '../../../shared/components/ErrorComponent';
 import '../studio.less';
+import { resourceWithoutMetadata } from './StudioContainer';
 
 const DASHBOARD_TYPE = 'StudioDashboard';
 
@@ -484,18 +485,18 @@ const WorkspaceMenu: React.FC<WorkspaceMenuProps> = ({
     data: TableResource | UnsavedTableResource
   ) => {
     if (selectedDashboard) {
-      const resource = await nexus.Resource.get(
+      const resource = (await nexus.Resource.get(
         orgLabel,
         projectLabel,
         encodeURIComponent(selectedDashboard['@id'])
-      );
+      )) as StudioResource;
       await nexus.Resource.update(
         orgLabel,
         projectLabel,
         encodeURIComponent(selectedDashboard['@id']),
         selectedDashboard._rev,
         {
-          ...resource,
+          ...resourceWithoutMetadata(resource),
           description: data.description,
           label: data['name'],
         }