🪟 🧹 Match sync catalog items by name and namespace, not their index (#…

…20387) * match by name and namespace, not index * streamline per PR review
airbytehq · Dec 13, 2022 · 7f6df3c · 7f6df3c
1 parent 2ea6110
commit 7f6df3c
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/airbyte-webapp/src/components/connection/CatalogTree/CatalogTree.tsx b/airbyte-webapp/src/components/connection/CatalogTree/CatalogTree.tsx
@@ -56,8 +56,13 @@ const CatalogTreeComponent: React.FC<React.PropsWithChildren<CatalogTreeProps>>
 
   const changedStreams = useMemo(
     () =>
-      streams.filter((stream, idx) => {
-        return stream.config?.selected !== initialValues.syncCatalog.streams[idx].config?.selected;
+      streams.filter((stream) => {
+        const matchingInitialValue = initialValues.syncCatalog.streams.find(
+          (initialStream) =>
+            initialStream.stream?.name === stream.stream?.name &&
+            initialStream.stream?.namespace === stream.stream?.namespace
+        );
+        return stream.config?.selected !== matchingInitialValue?.config?.selected;
       }),
     [initialValues.syncCatalog.streams, streams]
   );