marcel/familienarchiv
1
0
Fork 0
Code Issues 102 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(viewer): check res.ok on orphaned annotation DELETE to surface errors

Browse Source 
Without the guard, a failed DELETE (4xx/5xx) was silently swallowed and
annotationReloadKey was incremented anyway, leaving the annotation visible
and the user with no feedback. Now matches the deleteBlock() pattern
immediately above.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcel
2026-04-26 21:36:27 +02:00
parent b13c10936b
commit f22596a29d
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
frontend/src/routes/documents/[id]/+page.svelte
View File

@@ -120,7 +120,10 @@ async function handleAnnotationDeleteRequest(annotationId: string) {
await deleteBlock(block.id); await deleteBlock(block.id);
} else { } else {
// Annotation has no linked block — delete the annotation directly // Annotation has no linked block — delete the annotation directly
await fetch(`/api/documents/${doc.id}/annotations/${annotationId}`, { method: 'DELETE' }); const res = await fetch(`/api/documents/${doc.id}/annotations/${annotationId}`, {
method: 'DELETE'
});
if (!res.ok) throw new Error('Delete annotation failed');
annotationReloadKey++; annotationReloadKey++;
} }
} }