Skip to content

feat(admin): mandatory image processing and original/processed toggle#111

Merged
vgpastor merged 5 commits intomainfrom
feature/image-tab-mandatory-processing
Mar 9, 2026
Merged

feat(admin): mandatory image processing and original/processed toggle#111
vgpastor merged 5 commits intomainfrom
feature/image-tab-mandatory-processing

Conversation

@vgpastor
Copy link
Copy Markdown
Contributor

@vgpastor vgpastor commented Mar 9, 2026

Summary

  • Display processed images by default: Thumbnails show the processed (cropped/blurred/arrowed) version. A new toggle button in the lightbox and "Ver original" buttons on hover allow viewing the original image.
  • Auto-open processor on upload: When new images are uploaded, the image processor (crop/blur/arrow) opens automatically. Processing is mandatory — cancelling removes the image from the queue.
  • Block saving unprocessed images: Save button is disabled when unprocessed images exist. Warning banner and amber "Pendiente" badge provide clear feedback. The unprocessed addImages code path has been removed from handleSave.

Test plan

  • Open a DEA with processed images — verify thumbnails show processed version
  • Hover on an image with processed_url — verify "Ver original" button appears
  • Click lightbox toggle button — verify it switches between processed/original
  • Upload a new image — verify processor opens automatically
  • Cancel the processor — verify the image is removed from queue
  • Process an image, upload another — verify processor auto-opens for the next one
  • Try to save with unprocessed images — verify save button is disabled + warning shown
  • Process all images and save — verify both original and processed are stored

🤖 Generated with Claude Code

- Display processed images by default with toggle to view original in lightbox
- Auto-open image processor (crop/blur/arrow) when uploading new images
- Block saving unprocessed images (disabled save button + warning banner)
- Remove unprocessed addImages code path from handleSave
- Cancel processing removes unprocessed image from queue (mandatory flow)
- Add amber "Pendiente" badge for unprocessed new images

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Mar 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dea-map Ready Ready Preview, Comment Mar 9, 2026 7:22pm

Request Review

- Add newImageUrl support to process-image API (downloads from S3 instead of receiving base64)
- Frontend uploads files to S3 via /api/upload in background on file select
- Send S3 URL to process-image API instead of base64 data URL
- Add uploading overlay and disable save while uploads are in progress

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Compress images to 1920x1920 JPEG (quality 0.85, max 3MB) using existing
compressImageDataUrl utility before uploading to S3. Converts compressed
data URL to Blob for FormData upload. Prevents 413 Content Too Large
errors on both /api/upload and /api/admin/deas/[id]/process-image.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove sed/node version overwrites from release-mobile-app.yml
  (Android build.gradle + iOS project.pbxproj). Versions are now
  managed solely by propagate-version.js at dev time, not by CI.
- Add version-bump-check.js to migration-safety CI job (reuses
  existing base branch checkout).
- Disable automatic triggers on legacy build-android.yml and
  build-ios.yml to prevent double builds (kept as manual fallback).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ensure-tags.yml now creates both the git tag AND the GitHub Release
when a new mobile version is detected on push to main. The release
is created before triggering the build workflow, so it's visible
immediately.

The create-release job in release-mobile-app.yml is already
idempotent (skips if release exists), so no conflict.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vgpastor vgpastor merged commit e653f36 into main Mar 9, 2026
10 checks passed
@vgpastor vgpastor deleted the feature/image-tab-mandatory-processing branch March 9, 2026 19:26
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.

1 participant