Draft
Conversation
Adds delete_object support for removing S3 objects and a helper to list all keys under a prefix (for directory deletion). Stubs both in S3Stubbed for dev/testing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Adds DeleteSpec value object, factory, Deleter action, and a two-step confirmation view. Supports single file and directory deletion from both source and live buckets with cache busting. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
BulkUploader iterates over multiple UploadSpecs, delegating each to the existing Uploader with a shared output stream for unified progress. Adds from_form_params_bulk to UploadSpecFactory. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Adds routes for bulk-upload, delete, delete-dir, and their handlers. Updates browse view with bulk upload and delete links, view with a delete link, and upload form with conditional bulk mode. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Author
|
I'm not able to boot this in dev. I'm seeing |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/bulk-upload/*route with multi-file<input>that processes each file through the existingUploaderpipeline with unified streaming progressdeleteandall_keys_undermethods, with stubs for dev/testingAll new routes enforce the
static/prefix. Existing upload, edit, and browse functionality is untouched.QA
launch-devwith S3 stubbed): all new routes load without errors/bulk-upload/static/renders multi-file form, processes multiple files through pipeline/upload/static/still works identically (single file upload unchanged)/browse/static/shows bulk upload link, delete links on files and directories/view/static/...shows "Delete this image" link/delete/static/...shows confirmation page, POST deletes file from both buckets/delete-dir/static/.../lists affected files, POST deletes all from both bucketsconfirm()dialog fires before deletionstatic/prefix returns 400🤖 Generated with Claude Code