Skip to content

feat: improve bulk upload progress with server processing indicator - #239

Merged
trtajim merged 1 commit into
mainfrom
feat/bulk-upload-processing-phase
Aug 30, 2026
Merged

feat: improve bulk upload progress with server processing indicator#239
trtajim merged 1 commit into
mainfrom
feat/bulk-upload-processing-phase

Conversation

@trtajim

@trtajim trtajim commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Enhancements
    • Improved bulk image upload progress tracking.
    • Added clearer upload and server-processing states with updated progress messaging.
    • Added a spinner and status labels while images are being saved and processed.
    • Prevented the progress indicator from appearing complete before processing finishes.

@trtajim
trtajim merged commit a929b9a into main Aug 30, 2026
5 of 6 checks passed
@trtajim
trtajim deleted the feat/bulk-upload-processing-phase branch August 30, 2026 16:56
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 24bbfd77-c280-4e56-9877-35646ee08db9

📥 Commits

Reviewing files that changed from the base of the PR and between 538cbb3 and f6a065d.

📒 Files selected for processing (1)
  • resources/js/components/admin/BulkImageModal.vue

📝 Walkthrough

Walkthrough

The bulk image modal now separates upload progress from server-side processing. Upload progress stops below completion while the server processes images. The progress bar and submit button show dedicated processing states.

Changes

Bulk image processing

Layer / File(s) Summary
Processing state and progress lifecycle
resources/js/components/admin/BulkImageModal.vue
The modal adds isProcessingServer. Upload progress is capped at 90% during transfer and 95% when transfer completes. Lifecycle methods reset the processing state.
Processing status UI
resources/js/components/admin/BulkImageModal.vue
The progress bar shows processing text, a spinner, and animation during server processing. The submit button shows Saving to server....

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/bulk-upload-processing-phase

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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