fix(admin-web): correct 12 broken API calls across 5 pages

Pages fixed:
- waitlist: GET /waitlist/list → GET /waitlist (root)
- delivery: GET /delivery/log → GET /delivery/logs; disable retry button (no backend endpoint)
- reviews: GET /reviews/list → GET /reviews; approve/reject → POST /:id/decision with body; disable flag (no endpoint)
- jobs: GET /jobs/list → GET /jobs; GET /runs/list → GET /runs; trigger → POST /jobs/trigger with {jobId}
- gdpr-export: GET /exports/list → GET /exports; POST /exports/create → POST /exports

TODO Q1: delivery retry endpoint not implemented in backend
TODO Q2: reviews flag endpoint not implemented in backend
This commit is contained in:
saravanakumardb1 2026-03-21 21:06:49 -07:00
parent edf8926d6d
commit a3e94f3f3d
5 changed files with 30 additions and 36 deletions

View File

@ -3,10 +3,9 @@
import { createProxyFetch } from '@/lib/proxy-fetch';
import { useState, useEffect, useCallback } from 'react';
import { Mail, RotateCcw } from 'lucide-react';
import { Mail } from 'lucide-react';
import { Card, CardContent, CardHeader, CardTitle } from '@/components/ui/card';
import { Badge } from '@/components/ui/badge';
import { Button } from '@/components/ui/button';
import {
Table,
TableBody,
@ -63,7 +62,7 @@ export default function DeliveryPage() {
const params = new URLSearchParams();
if (statusFilter !== 'all') params.set('status', statusFilter);
const qs = params.toString() ? `?${params.toString()}` : '';
const data = await apiFetch(`log${qs}`);
const data = await apiFetch(`logs${qs}`);
setEntries(Array.isArray(data?.entries) ? data.entries : Array.isArray(data) ? data : []);
setLoading(false);
}, [statusFilter]);
@ -72,10 +71,12 @@ export default function DeliveryPage() {
void loadData();
}, [loadData]);
async function handleRetry(id: string) {
await apiFetch(`log/${id}/retry`, { method: 'POST' });
loadData();
}
// TODO Q1: Backend has no retry endpoint for delivery log entries.
// When /delivery/logs/:id/retry is implemented, uncomment this.
// async function handleRetry(id: string) {
// await apiFetch(`logs/${id}/retry`, { method: 'POST' });
// loadData();
// }
const deliveredCount = entries.filter(e => e.status === 'delivered').length;
const failedCount = entries.filter(e => e.status === 'failed').length;
@ -190,17 +191,7 @@ export default function DeliveryPage() {
{formatDate(e.createdAt)}
</TableCell>
<TableCell>
{e.status === 'failed' && (
<Button
variant="ghost"
size="icon"
className="h-7 w-7"
onClick={() => handleRetry(e.id)}
title="Retry"
>
<RotateCcw className="h-3.5 w-3.5" />
</Button>
)}
{/* TODO Q1: Retry button disabled — backend has no retry endpoint yet */}
</TableCell>
</TableRow>
);

View File

@ -39,7 +39,7 @@ export default function GdprExportPage() {
if (!userId.trim()) return;
setLoading(true);
try {
const res = await fetch(`/api/exports/list?userId=${encodeURIComponent(userId)}`, {
const res = await fetch(`/api/exports?userId=${encodeURIComponent(userId)}`, {
headers: { 'Content-Type': 'application/json' },
});
const data = await res.json();
@ -55,7 +55,7 @@ export default function GdprExportPage() {
setExporting(true);
setExportResult(null);
try {
const res = await fetch('/api/exports/create', {
const res = await fetch('/api/exports', {
method: 'POST',
headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({ userId: userId.trim(), entityTypes: ['all'], format: 'json' }),

View File

@ -70,7 +70,7 @@ export default function JobsPage() {
const loadData = useCallback(async () => {
setLoading(true);
const [jData, rData] = await Promise.all([jobsFetch('list'), runsFetch('list?limit=50')]);
const [jData, rData] = await Promise.all([jobsFetch(''), runsFetch('?limit=50')]);
setJobs(Array.isArray(jData?.jobs) ? jData.jobs : Array.isArray(jData) ? jData : []);
setRuns(Array.isArray(rData?.runs) ? rData.runs : Array.isArray(rData) ? rData : []);
setLoading(false);
@ -81,7 +81,7 @@ export default function JobsPage() {
}, [loadData]);
async function handleTrigger(id: string) {
await jobsFetch(`${id}/trigger`, { method: 'POST' });
await jobsFetch('trigger', { method: 'POST', body: JSON.stringify({ jobId: id }) });
loadData();
}

View File

@ -62,7 +62,7 @@ export default function ReviewsPage() {
const params = new URLSearchParams();
if (statusFilter !== 'all') params.set('status', statusFilter);
const qs = params.toString() ? `?${params.toString()}` : '';
const data = await apiFetch(`list${qs}`);
const data = await apiFetch(`${qs}`);
setReviews(Array.isArray(data?.reviews) ? data.reviews : Array.isArray(data) ? data : []);
setTotal(data?.total ?? (Array.isArray(data?.reviews) ? data.reviews.length : 0));
setLoading(false);
@ -73,19 +73,27 @@ export default function ReviewsPage() {
}, [loadData]);
async function handleApprove(id: string) {
await apiFetch(`${id}/approve`, { method: 'POST' });
await apiFetch(`${id}/decision`, {
method: 'POST',
body: JSON.stringify({ decision: 'approved' }),
});
loadData();
}
async function handleReject(id: string) {
await apiFetch(`${id}/reject`, { method: 'POST' });
await apiFetch(`${id}/decision`, {
method: 'POST',
body: JSON.stringify({ decision: 'rejected' }),
});
loadData();
}
async function handleFlag(id: string) {
await apiFetch(`${id}/flag`, { method: 'POST' });
loadData();
}
// TODO Q2: Backend has no /reviews/:id/flag endpoint.
// When flagging is implemented, uncomment this.
// async function handleFlag(id: string) {
// await apiFetch(`${id}/flag`, { method: 'POST' });
// loadData();
// }
const pendingCount = reviews.filter(r => r.status === 'pending').length;
const flaggedCount = reviews.filter(r => r.flagged).length;
@ -245,12 +253,7 @@ export default function ReviewsPage() {
Reject
</DropdownMenuItem>
)}
{!r.flagged && (
<DropdownMenuItem onClick={() => handleFlag(r.id)}>
<Flag className="mr-2 h-4 w-4" />
Flag
</DropdownMenuItem>
)}
{/* TODO Q2: Flag button disabled — no backend endpoint yet */}
</DropdownMenuContent>
</DropdownMenu>
</TableCell>

View File

@ -85,7 +85,7 @@ export default function WaitlistPage() {
if (statusFilter !== 'all') params.set('status', statusFilter);
if (search) params.set('search', search);
const qs = params.toString() ? `?${params.toString()}` : '';
const data = await apiFetch(`list${qs}`);
const data = await apiFetch(`${qs}`);
if (data?.entries) {
setEntries(data.entries);
setTotal(data.total ?? data.entries.length);