Merge pull request #37 from EyeSeeTea/fix/migrations

Fix deleted sections on migrations executed
EyeSeeTea · Jul 3, 2024 · 99ab1bc · 99ab1bc
2 parents 2ade2d5 + 66e93fc
commit 99ab1bc
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/migrations/tasks/02.remove-duplicated-nodes.ts b/src/migrations/tasks/02.remove-duplicated-nodes.ts
@@ -16,8 +16,11 @@ class RemoveDuplicatedNodesMigration {
             const fixedTrees = landingTrees.map(tree =>
                 tree.reduce<PersistedLandingPage>((acc, node) => {
                     const inParentTree =
-                        node.parent === "none" || node.type === "root" || acc.some(parent => parent.id === node.parent);
-                    const alreadyExist = acc.some(n => n.id === node.id);
+                        node.parent === "none" ||
+                        node.type === "root" ||
+                        tree.some(maybeParent => maybeParent.id === node.parent); // To avoid nodes out of place
+
+                    const alreadyExist = acc.some(n => n.id === node.id); // To avoid duplicated nodes
                     if (inParentTree && !alreadyExist) {
                         return [...acc, node];
                     } else {