From 70272d1b1ff9a4c1fc0474ebe9b44ab1fa97b55e Mon Sep 17 00:00:00 2001
From: Lucano Vera <lucanovera@live.com.ar>
Date: Tue, 17 Dec 2024 16:45:28 -0300
Subject: [PATCH] LA-56 Only trigger dataset save if values have changed
 (#5612)

Co-authored-by: Lucano Vera <lucanovera@ethyca.com>
---
 CHANGELOG.md                                                  | 1 +
 .../system_portal_config/forms/ConnectorParameters.tsx        | 4 +++-
 .../forms/fields/DatasetConfigField/useDatasetConfigField.tsx | 3 ++-
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index ca8124523e..d1e8deb8f8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -30,6 +30,7 @@ The types of changes are:
 
 ### Fixed
 - Fixing quickstart.py script [#5585](https://github.com/ethyca/fides/pull/5585)
+- Removed unnecessary double notification when updating database integrations [#5612](https://github.com/ethyca/fides/pull/5612)
 
 ## [2.51.2](https://github.com/ethyca/fides/compare/2.51.1...2.51.2)
 
diff --git a/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/ConnectorParameters.tsx b/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/ConnectorParameters.tsx
index 7697b755e2..7027c56080 100644
--- a/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/ConnectorParameters.tsx
+++ b/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/ConnectorParameters.tsx
@@ -295,7 +295,9 @@ export const useConnectorForm = ({
         values.dataset &&
         connectionOption.type === SystemType.DATABASE
       ) {
-        await patchConnectionDatasetConfig(values, connectionConfig.key);
+        await patchConnectionDatasetConfig(values, connectionConfig.key, {
+          showSuccessAlert: false,
+        });
       }
 
       successAlert(
diff --git a/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/fields/DatasetConfigField/useDatasetConfigField.tsx b/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/fields/DatasetConfigField/useDatasetConfigField.tsx
index 2b1bbc96f6..c82ad57fb2 100644
--- a/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/fields/DatasetConfigField/useDatasetConfigField.tsx
+++ b/clients/admin-ui/src/features/datastore-connections/system_portal_config/forms/fields/DatasetConfigField/useDatasetConfigField.tsx
@@ -50,6 +50,7 @@ export const useDatasetConfigField = ({
   const patchConnectionDatasetConfig = async (
     values: ConnectionConfigFormValues,
     connectionConfigKey: string,
+    { showSuccessAlert = true }: { showSuccessAlert?: boolean } = {},
   ) => {
     const newDatasetPairs: DatasetConfigCtlDataset[] =
       values.dataset?.map((datasetKey) => ({
@@ -65,7 +66,7 @@ export const useDatasetConfigField = ({
     const payload = await putDatasetConfig(params).unwrap();
     if (payload.failed?.length > 0) {
       errorAlert(payload.failed[0].message);
-    } else {
+    } else if (showSuccessAlert) {
       successAlert("Dataset successfully updated!");
     }
   };