Skip to content

fix: avoid oversized web multi-delete requests#273

Merged
uxjulia merged 1 commit into
developmentfrom
fix/web-multi-delete-limit
Jun 14, 2026
Merged

fix: avoid oversized web multi-delete requests#273
uxjulia merged 1 commit into
developmentfrom
fix/web-multi-delete-limit

Conversation

@uxjulia

@uxjulia uxjulia commented Jun 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • Change the web file manager multi-delete flow to send one small delete request per selected item instead of one large paths payload.
  • Keep the delete modal disabled while the batch is running and report partial failures without aborting the whole selection.
  • Add an Unreleased changelog entry for the user-facing fix.

Root Cause

The browser was posting all selected paths as one URL-encoded JSON array to /delete. With larger selections, that request can exceed the small request/parsing budget on the ESP32-C3 before the firmware deletes anything. Splitting the batch in the browser keeps each request small and reuses the existing single-path delete handler.

Fixes #269.

Validation

  • python3 scripts/build_web.py
  • pio run -e simulator
  • pio run -e tiny -j1
  • git diff --check

@uxjulia uxjulia marked this pull request as ready for review June 14, 2026 18:26
@uxjulia uxjulia linked an issue Jun 14, 2026 that may be closed by this pull request
@uxjulia uxjulia merged commit bdcc6e5 into development Jun 14, 2026
5 checks passed
@uxjulia uxjulia deleted the fix/web-multi-delete-limit branch June 14, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web UI fails when attempting more than 9 multiple deletes

1 participant