fix: broken feature after merge

datsmysong · Apr 9, 2024 · 640df22 · 640df22
1 parent 8448fa1
commit 640df22
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 8 deletions.
diff --git a/backend/src/socketio/Room.ts b/backend/src/socketio/Room.ts
@@ -120,7 +120,7 @@ export default class Room {
 
     this.queue.push({
       ...track,
-      addedBy: "TODO", // TODO: Add the id of the user who added the track
+      addedBy: accountId,
       votes: [],
     });
     if (this.queue.length !== 1) return;

diff --git a/commons/backend-types.ts b/commons/backend-types.ts
@@ -8,16 +8,17 @@ export interface JSONTrack {
   /** An array of all genres of this song */
   genres: string[];
   id: string;
-  votes?: string[]; // array of user ids
-  addedBy: string;
 }
 
 export interface RoomJSONTrack extends JSONTrack {
   /**
    * The user id of the user who added the track to the queue
    */
   addedBy: string;
-  votes: string[]; // array of user ids
+  /**
+   * An array of user ids of users who have voted to skip this track
+   */
+  votes: string[];
 }
 
 export interface PlayingJSONTrack extends RoomJSONTrack {

diff --git a/expo/app/(tabs)/rooms/[id]/add.tsx b/expo/app/(tabs)/rooms/[id]/add.tsx
@@ -33,7 +33,11 @@ export default function AddTrack() {
         "Les utilisateurs anonymes ne sont pas autorisés à accéder à cette fonctionnalité. Veuillez vous connecter."
       );
     try {
-      socket.emit("queue:add", new URL(url).toString(), userProfile.account_id);
+      socket.emit(
+        "queue:add",
+        new URL(url).toString(),
+        userProfile.user_profile_id
+      );
     } catch {
       setSearchBar("");
       setResult(null);

diff --git a/expo/components/room/TrackItem.tsx b/expo/components/room/TrackItem.tsx
@@ -19,7 +19,6 @@ export default function TrackItem(prop: {
   handleDislike: () => void;
   disliked: boolean;
   addedBy: string;
-  accountId: string;
 }) {
   const {
     title,
@@ -48,8 +47,8 @@ export default function TrackItem(prop: {
       artistsName={artists}
       imgUrl={rawImageUrl}
       profilePictureImage={
-        prop.accountId ? (
-          <Avatar id={prop.accountId} style={itemStyles.profileImage} />
+        prop.addedBy ? (
+          <Avatar id={prop.addedBy} style={itemStyles.profileImage} />
         ) : (
           <Image
             source={require("../../assets/images/album-cover.jpg")}