fix(frontend): confirm asset deletion and serialize reference uploads

↗ view on GitHub · Amal · 2026-08-12 · baba1840

WHY THIS MATTERS
Reference-file Delete acted immediately from a row menu - one misclick
permanently destroyed a file, while every other destructive action in this
feature (workflow delete, column delete) goes through ConfirmPopup.
Uploads had a matching gap: the toolbar button disabled during an upload
but the drag-drop path forwarded drops unconditionally, interleaving
batches and clobbering the shared warning text.

HOW IT WORKS
Delete now stages the file behind the same ConfirmPopup pattern as
workflow deletion. Uploads take a synchronous in-flight guard (a ref, so a
second drop in the same render frame is still caught), the detail page
stops accepting drops while a batch runs, and warnings append instead of
overwrite so an unsupported-file notice survives a later failure.

Found by review of PR #309.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E4PXCdenNH5Mqhm5Sre9Zs
Repository open-legal-products/mike
Author Amal <mamalanand3@gmail.com>
Authored
Committed
Parents 2c1b6214
Stats 2 files changed , +69 , -4
Part of Refactor workflows into defaults, add-ons, and quick actions

Capture this commit into my fork

Download a Markdown prompt that tells Claude how to port this exact commit into your working tree. Run it via claude -p < capture-commit-baba1840.md from inside the repo you want the change in.

⬇ Download capture-commit-baba1840.md