From 7fe146f97ddd3f5a8d0c1a996a73cb296c28b9cc Mon Sep 17 00:00:00 2001 From: Pinapelz Date: Mon, 7 Jul 2025 01:07:34 -0700 Subject: implement score deletion --- .../components/displays/DancerushScoreDisplay.tsx | 25 ++++++++++++++++- .../components/displays/GenericScoreDisplay.tsx | 25 ++++++++++++++++- frontend/src/pages/Game.tsx | 0 frontend/src/pages/Score.tsx | 32 ++++++++++++++++++++-- frontend/src/types/constants.ts | 2 +- 5 files changed, 79 insertions(+), 5 deletions(-) delete mode 100644 frontend/src/pages/Game.tsx (limited to 'frontend') diff --git a/frontend/src/components/displays/DancerushScoreDisplay.tsx b/frontend/src/components/displays/DancerushScoreDisplay.tsx index e99f6e9..b030db7 100644 --- a/frontend/src/components/displays/DancerushScoreDisplay.tsx +++ b/frontend/src/components/displays/DancerushScoreDisplay.tsx @@ -15,6 +15,7 @@ interface ScoreDisplayProps { sortField: string; sortDirection: "asc" | "desc"; onSort: (field: string) => void; + onDelete?: (scoreId: number) => void; } const DancerushScoreDisplay: React.FC = ({ @@ -23,6 +24,7 @@ const DancerushScoreDisplay: React.FC = ({ sortField, sortDirection, onSort, + onDelete, }) => { // Key mappings for better display names. Hit or miss const keyDisplayNames: Record = { @@ -242,6 +244,9 @@ const DancerushScoreDisplay: React.FC = ({ "timestamp", ].filter((key) => allKeys.includes(key)); + // Add actions column if delete function is provided + const showActions = onDelete && viewMode === "table"; + if (scores.length === 0) { return (
@@ -374,13 +379,18 @@ const DancerushScoreDisplay: React.FC = ({ )} ))} + {showActions && ( + + Actions + + )} {sortedScores.map((score, index) => ( {tableKeys.map((key) => ( @@ -411,6 +421,19 @@ const DancerushScoreDisplay: React.FC = ({ )} ))} + {showActions && ( + + + + )} ))} diff --git a/frontend/src/components/displays/GenericScoreDisplay.tsx b/frontend/src/components/displays/GenericScoreDisplay.tsx index c30e475..45bf1ca 100644 --- a/frontend/src/components/displays/GenericScoreDisplay.tsx +++ b/frontend/src/components/displays/GenericScoreDisplay.tsx @@ -13,6 +13,7 @@ interface ScoreDisplayProps { sortField: string; sortDirection: "asc" | "desc"; onSort: (field: string) => void; + onDelete?: (scoreId: number) => void; } const ScoreDisplay: React.FC = ({ @@ -21,6 +22,7 @@ const ScoreDisplay: React.FC = ({ sortField, sortDirection, onSort, + onDelete, }) => { // Key mappings for better display names. Hit or miss const keyDisplayNames: Record = { @@ -238,6 +240,9 @@ const ScoreDisplay: React.FC = ({ "timestamp", ].filter((key) => allKeys.includes(key)); + // Add actions column if delete function is provided + const showActions = onDelete && viewMode === "table"; + if (scores.length === 0) { return (
@@ -369,13 +374,18 @@ const ScoreDisplay: React.FC = ({ )} ))} + {showActions && ( + + Actions + + )} {sortedScores.map((score, index) => ( {tableKeys.map((key) => ( @@ -406,6 +416,19 @@ const ScoreDisplay: React.FC = ({ )} ))} + {showActions && ( + + + + )} ))} diff --git a/frontend/src/pages/Game.tsx b/frontend/src/pages/Game.tsx deleted file mode 100644 index e69de29..0000000 diff --git a/frontend/src/pages/Score.tsx b/frontend/src/pages/Score.tsx index e32f001..0d18d7a 100644 --- a/frontend/src/pages/Score.tsx +++ b/frontend/src/pages/Score.tsx @@ -5,7 +5,6 @@ import { NavBar } from "../components/NavBar"; import SessionExpiredPopup from "../components/SessionExpiredPopup"; import ScoreDisplay from "../components/displays/GenericScoreDisplay"; import DancerushScoreDisplay from "../components/displays/DancerushScoreDisplay"; -// TODO: selector for PB/Recent type SortField = string; type SortDirection = "asc" | "desc"; @@ -79,7 +78,7 @@ const Score = () => { url.searchParams.append("sortKey", requestOrder); url.searchParams.append("direction", "asc"); - const response = await fetch(url.toString()); + const response = await fetch(url.toString(), {credentials: 'include'}); if (!response.ok) throw new Error("Failed to fetch scores"); const data = await response.json(); const flattened = data.scores.map(flattenScoreData); @@ -96,6 +95,33 @@ const Score = () => { [user, gameName, requestOrder], ); + const handleDeleteScore = async (scoreId: number) => { + if (!user) return; + + if (!confirm("Are you sure you want to delete this score? This action cannot be undone.")) { + return; + } + + try { + const url = new URL(import.meta.env.VITE_API_URL + "/scores"); + url.searchParams.append("userId", user.id); + url.searchParams.append("internalGameName", gameName); + url.searchParams.append("scoreId", scoreId.toString()); + + const response = await fetch(url.toString(), { + method: "DELETE", + credentials: "include", + }); + + if (!response.ok) throw new Error("Failed to delete score"); + + await fetchScores(currentPage); + } catch (error) { + console.error("Failed to delete score:", error); + alert("Failed to delete score. Please try again."); + } + }; + useEffect(() => { if (user) fetchScores(1); }, [user, fetchScores]); @@ -175,6 +201,7 @@ const Score = () => { sortField={sortField} sortDirection={sortDirection} onSort={handleSort} + onDelete={handleDeleteScore} /> ); default: @@ -185,6 +212,7 @@ const Score = () => { sortField={sortField} sortDirection={sortDirection} onSort={handleSort} + onDelete={handleDeleteScore} /> ); } diff --git a/frontend/src/types/constants.ts b/frontend/src/types/constants.ts index b1309e9..028545d 100644 --- a/frontend/src/types/constants.ts +++ b/frontend/src/types/constants.ts @@ -17,7 +17,7 @@ export function getFilterOptions(game: string): { value: string; label: string } switch (game) { case "dancerush": return [ - { value: "timestamp", label: "Recent" }, + { value: "timestamp", label: "Recently Played" }, { value: "score", label: "Score" }, { value: "lamp", label: "Rank" }, { value: "lamp_diff", label: "Difficulty"} -- cgit v1.2.3