fix: use toSorted() to copy & sort instead of slice().sort() (#3247)

theopensystemslab · Jun 5, 2024 · dede865 · dede865
1 parent 0329997
commit dede865
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/editor.planx.uk/src/pages/Team.tsx b/editor.planx.uk/src/pages/Team.tsx
@@ -286,13 +286,11 @@ const Team: React.FC = () => {
       .getFlows(teamId)
       .then((res: { flows: any[] }) => {
         // Copy the array and sort by most recently edited desc using last associated operation.createdAt, not flow.updatedAt
-        const sortedFlows = res.flows
-          .slice()
-          .sort((a, b) =>
-            b.operations[0]["createdAt"].localeCompare(
-              a.operations[0]["createdAt"],
-            ),
-          );
+        const sortedFlows = res.flows.toSorted((a, b) =>
+          b.operations[0]["createdAt"].localeCompare(
+            a.operations[0]["createdAt"],
+          ),
+        );
         setFlows(sortedFlows);
       });
   }, [teamId, setFlows]);