2314: sort playlists based on weight (or else drag and drop does not …

…make any sense)
os2display · Oct 23, 2024 · f10897b · f10897b
1 parent 11bbeb1
commit f10897b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,7 @@ All notable changes to this project will be documented in this file.
   - Change bootstrap column class from `col-md-8` -> `col-md-12` 
   - update api.generated.ts to match [related pr](https://github.com/os2display/display-api-service/pull/213)
   - Add @rtk-incubator/rtk-query-codegen-openapi to package.json in `src/redux/api`
+  - Sort playlists based on weight in drag/drop component
 
 ## [2.1.0] - 2024-10-23
 

diff --git a/src/components/playlist-drag-and-drop/playlist-drag-and-drop.jsx b/src/components/playlist-drag-and-drop/playlist-drag-and-drop.jsx
@@ -77,12 +77,14 @@ function PlaylistDragAndDrop({
     if (selectedPlaylistsByRegion) {
       setTotalItems(selectedPlaylistsByRegion["hydra:totalItems"]);
       const newPlaylists = selectedPlaylistsByRegion["hydra:member"].map(
-        ({ playlist }) => {
-          return playlist;
-        }
+        ({ playlist, weight }) => ({ ...playlist, weight })
       );
 
-      setSelectedData([...selectedData, ...newPlaylists]);
+      const selected = [...selectedData, ...newPlaylists].sort(
+        (a, b) => a.weight - b.weight
+      );
+
+      setSelectedData(selected);
       callbackToinitializePlaylists(selectedPlaylistsByRegion);
     }
   }, [selectedPlaylistsByRegion]);