feat(frontend): add batch pause/resume + confirmation modal to StreamsTable bulk actions (#762) - #941
Conversation
|
@utosjr1913 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
@utosjr1913 is attempting to deploy a commit to the ritik4ever's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 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. Comment |
Summary
Closes #762.
Bulk selection (checkboxes, select all/deselect all, sequential batch cancel) was
already implemented in
StreamsTable.tsx. This PR adds the three pieces the issuestill required:
behind an explicit confirm step
console.errorChanges
frontend/src/components/BulkActionModal.tsx— confirm → running →complete modal, reusing the app's existing
.modal-backdrop/.modal-panelstyles and
useFocusTraphook (same pattern as the existingClaimBatchModal.tsx).frontend/src/components/StreamsTable.tsx:isStreamSelectablenow includespausedstreams (needed so Resume haseligible targets)
BulkActionBarnow renders Pause / Resume / Cancel, all routed through thenew confirmation modal instead of firing immediately
requestBulkAction/runBulkAction, which collects{ streamId, message }failures per streamfrontend/src/index.css: added the.bulk-action-barstyles, which werereferenced in the component but never defined anywhere — the floating action
bar has been rendering completely unstyled until this PR.
BULK_SELECTION_VISUAL_GUIDE.mdandBULK_SELECTION_IMPLEMENTATION.mdto match.
Verification against acceptance criteria
pause/resume too.
Known blocker for CI verification
frontend/package.jsonis empty in this repo (pre-existing, unrelated) —npm installfails immediately, so I couldn't run the test suite locally.Flagging for a separate fix.