fix patch request for inherited ownership (open-metadata#16383)

gorgias · May 22, 2024 · c6108cf · c6108cf
1 parent f5edf6c
commit c6108cf
Show file tree

Hide file tree

Showing 9 changed files with 9 additions and 34 deletions.
diff --git a/...ain/resources/ui/src/components/Dashboard/DashboardDetails/DashboardDetails.component.tsx b/...ain/resources/ui/src/components/Dashboard/DashboardDetails/DashboardDetails.component.tsx
@@ -265,7 +265,7 @@ const DashboardDetails = ({
     async (newOwner?: Dashboard['owner']) => {
       const updatedDashboard = {
         ...dashboardDetails,
-        owner: newOwner ? { ...owner, ...newOwner } : undefined,
+        owner: newOwner,
       };
       await onDashboardUpdate(updatedDashboard, 'owner');
     },

diff --git a/...ui/src/main/resources/ui/src/components/MlModel/MlModelDetail/MlModelDetail.component.tsx b/...ui/src/main/resources/ui/src/components/MlModel/MlModelDetail/MlModelDetail.component.tsx
@@ -185,12 +185,7 @@ const MlModelDetail: FC<MlModelDetailProp> = ({
     async (newOwner?: Mlmodel['owner']) => {
       const updatedMlModelDetails = {
         ...mlModelDetail,
-        owner: newOwner
-          ? {
-              ...mlModelDetail.owner,
-              ...newOwner,
-            }
-          : undefined,
+        owner: newOwner,
       };
       await settingsUpdateHandler(updatedMlModelDetails);
     },

diff --git a/...c/main/resources/ui/src/components/Pipeline/PipelineDetails/PipelineDetails.component.tsx b/...c/main/resources/ui/src/components/Pipeline/PipelineDetails/PipelineDetails.component.tsx
@@ -225,7 +225,7 @@ const PipelineDetails = ({
     async (newOwner?: Pipeline['owner']) => {
       const updatedPipelineDetails = {
         ...pipelineDetails,
-        owner: newOwner ? { ...owner, ...newOwner } : undefined,
+        owner: newOwner,
       };
       await settingsUpdateHandler(updatedPipelineDetails);
     },

diff --git a/...ata-ui/src/main/resources/ui/src/components/Topic/TopicDetails/TopicDetails.component.tsx b/...ata-ui/src/main/resources/ui/src/components/Topic/TopicDetails/TopicDetails.component.tsx
@@ -209,12 +209,7 @@ const TopicDetails: React.FC<TopicDetailsProps> = ({
     async (newOwner?: Topic['owner']) => {
       const updatedTopicDetails = {
         ...topicDetails,
-        owner: newOwner
-          ? {
-              ...owner,
-              ...newOwner,
-            }
-          : undefined,
+        owner: newOwner,
       };
       await onTopicUpdate(updatedTopicDetails, 'owner');
     },

diff --git a/openmetadata-ui/src/main/resources/ui/src/pages/DatabaseDetailsPage/DatabaseDetailsPage.tsx b/openmetadata-ui/src/main/resources/ui/src/pages/DatabaseDetailsPage/DatabaseDetailsPage.tsx
@@ -303,7 +303,7 @@ const DatabaseDetails: FunctionComponent = () => {
     async (owner: Database['owner']) => {
       const updatedData = {
         ...database,
-        owner: owner ? { ...database?.owner, ...owner } : undefined,
+        owner: owner,
       };
 
       await settingsUpdateHandler(updatedData as Database);

diff --git a/...ta-ui/src/main/resources/ui/src/pages/DatabaseSchemaPage/DatabaseSchemaPage.component.tsx b/...ta-ui/src/main/resources/ui/src/pages/DatabaseSchemaPage/DatabaseSchemaPage.component.tsx
@@ -322,7 +322,7 @@ const DatabaseSchemaPage: FunctionComponent = () => {
       try {
         const updatedData = {
           ...databaseSchema,
-          owner: owner ? { ...databaseSchema?.owner, ...owner } : undefined,
+          owner: owner,
         };
 
         const response = await saveUpdatedDatabaseSchemaData(

diff --git a/...data-ui/src/main/resources/ui/src/pages/SearchIndexDetailsPage/SearchIndexDetailsPage.tsx b/...data-ui/src/main/resources/ui/src/pages/SearchIndexDetailsPage/SearchIndexDetailsPage.tsx
@@ -287,12 +287,7 @@ function SearchIndexDetailsPage() {
       }
       const updatedSearchIndexDetails = {
         ...searchIndexDetails,
-        owner: newOwner
-          ? {
-              ...owner,
-              ...newOwner,
-            }
-          : undefined,
+        owner: newOwner,
       };
       await onSearchIndexUpdate(updatedSearchIndexDetails, 'owner');
     },

diff --git a/openmetadata-ui/src/main/resources/ui/src/pages/StoredProcedure/StoredProcedurePage.tsx b/openmetadata-ui/src/main/resources/ui/src/pages/StoredProcedure/StoredProcedurePage.tsx
@@ -325,12 +325,7 @@ const StoredProcedurePage = () => {
       }
       const updatedEntityDetails = {
         ...storedProcedure,
-        owner: newOwner
-          ? {
-              ...owner,
-              ...newOwner,
-            }
-          : undefined,
+        owner: newOwner,
       };
       await handleStoreProcedureUpdate(updatedEntityDetails, 'owner');
     },

diff --git a/openmetadata-ui/src/main/resources/ui/src/pages/TableDetailsPageV1/TableDetailsPageV1.tsx b/openmetadata-ui/src/main/resources/ui/src/pages/TableDetailsPageV1/TableDetailsPageV1.tsx
@@ -406,12 +406,7 @@ const TableDetailsPageV1: React.FC = () => {
       }
       const updatedTableDetails = {
         ...tableDetails,
-        owner: newOwner
-          ? {
-              ...owner,
-              ...newOwner,
-            }
-          : undefined,
+        owner: newOwner,
       };
       await onTableUpdate(updatedTableDetails, 'owner');
     },