Changeset 1579b4f for frontend/src/pages/SongDetail.tsx

Timestamp:

02/10/26 21:58:39 (5 months ago)

Author:

Filip Gavrilovski <filipgavrilovski28@…>

Branches:

main

Children:

92db381

Parents:

0808ef2

Message:

refactor artist, collection, listener components to have similar style as other components; refactor some dto-s

File:

• frontend/src/pages/SongDetail.tsx (modified) (4 diffs)

frontend/src/pages/SongDetail.tsx

@@ -1 +1 @@
 import { useEffect, useRef, useState } from "react";
 import { Link, useParams } from "react-router-dom";
+import { toast } from "react-toastify";
 import axiosInstance from "../api/axiosInstance";
 import { useAuth } from "../context/authContext";
 import { usePlayer } from "../context/playerContext";
 import type { SongDetail as SongDetailType } from "../utils/types";
+import { toEmbedUrl } from "../utils/utils";
 
 const ROLE_LABELS: Record<string, string> = {
@@ -18 +20 @@
 const formatRole = (role: string): string => {
         return ROLE_LABELS[role] || role.replace(/_/g, " ");
-};
-
-// convert a regular youtube URL to an embeddable URL
-const toEmbedUrl = (url: string): string => {
-        try {
-                const parsed = new URL(url);
-                // youtube.com/watch?v=ID
-                if (
-                        (parsed.hostname === "www.youtube.com" ||
-                                parsed.hostname === "youtube.com") &&
-                        parsed.searchParams.has("v")
-                ) {
-                        return `https://www.youtube.com/embed/${parsed.searchParams.get("v")}`;
-                }
-                // youtu.be/ID
-                if (parsed.hostname === "youtu.be") {
-                        return `https://www.youtube.com/embed${parsed.pathname}`;
-                }
-                // already an embed URL or other provider – return as-is
-                return url;
-        } catch {
-                return url;
-        }
 };
 
@@ -137 +116 @@
                         setSong(response.data);
                 } catch (err) {
-                        // todo :add toast
+                        toast.error("Failed to delete review");
                         console.error("Error deleting review:", err);
                 }
@@ -152 +131 @@
                         );
                 } catch (err) {
+                        toast.error("Failed to toggle like");
                         console.error("Error toggling like:", err);
                 }