←
TRCKR-504
Created: 2025-12-18
•
Updated: 2025-12-18
Relationships
Loading...
Attachments
Loading...
Comments (3)
qa-agent
·
2025-12-18
## QA PASSED
See QA report document: 2f47e6ed-5feb-4dd1-82ee-edba34e45744
**Tests Performed:**
- Basic bulk delete (single comment) - PASS
- Bulk delete (multiple comments) - PASS
- Empty list validation - PASS
- Invalid/non-existent IDs handling - PASS
- Mixed valid/invalid IDs - PASS
- Single delete regression tests - PASS
- 93 unit tests - PASS
**Minor Issue Found:**
Invalid action types return misleading 'Deleted 0' message instead of error. Non-blocking - comments are protected.
All hypotheses tested. Fix verified working.
agent
·
2025-12-18
## Implementation Complete
### Changes Made
1. **Backend endpoint** (server/routes/web.py:2646-2688):
- Added POST /web/comments/bulk endpoint
- Supports delete action for bulk comment deletion
- Returns success/error toast messages via HTMLResponse script
2. **Frontend bulk toolbar** (server/templates/comments.html:7-9):
- Added bulk toolbar macro with delete action
- Uses use_entity_ids=true for proper ID generation
3. **JavaScript functions** (server/templates/base.html:4455-4587):
- Enhanced updateBulkToolbarComments() to show/hide toolbar and update count
- Added clearBulkSelectionComments() to clear checkbox selection
- Added onCommentBulkActionChange() to handle action selection UI
- Added executeBulkActionComments() to execute bulk delete with confirmation
4. **Bug fix** (server/routes/web.py:2636):
- Added authors_list context variable for partial template compatibility
5. **E2E tests** (tests/e2e/test_comments_bulk.py):
- TestCommentBulkDelete: Tests for single/multiple delete, cancel behavior
- TestCommentBulkToolbar: Tests for toolbar visibility, clear selection, select all
### Test Results
- All 1045 existing tests pass
- 8 new E2E tests created (require Docker to run)
### Verification
The bulk delete feature follows the same pattern as other entities (tags, documents, attachments).
worker-batch2-groupC-3
·
2025-12-18
Assigned for Batch 2 Group C - Bulk actions: comments