Conversation
|
Review changes with Changed Files
|
lucaspar
added
gateway
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request implements a "Quick Add Capture(s) to Dataset" feature that allows users to add one or multiple captures to a dataset through two workflows: (1) a selection mode in the captures table where users can select multiple captures via checkboxes, and (2) planned support for single-capture quick-add from dropdown menus (though the UI for this is not yet implemented in the templates).
Changes:
- Added backend API endpoints for quick-adding captures to datasets and retrieving eligible datasets
- Implemented frontend selection mode with checkbox-based multi-select in the captures table
- Created a modal interface for selecting the target dataset and confirming the operation
- Added support for automatic multi-channel capture grouping during addition
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| gateway/sds_gateway/users/views.py | Added QuickAddCapturesToDatasetView for adding captures and UserDatasetsForQuickAddView for listing eligible datasets with proper permission checks |
| gateway/sds_gateway/users/urls.py | Registered two new URL patterns for the quick-add API endpoints |
| gateway/sds_gateway/templates/users/partials/quick_add_to_dataset_modal.html | New modal template for dataset selection with accessible form controls |
| gateway/sds_gateway/templates/users/partials/captures_page_table.html | Added selection column with checkboxes (hidden by default, shown in selection mode) |
| gateway/sds_gateway/templates/users/file_list.html | Integrated selection mode buttons and quick-add modal into the file list page |
| gateway/sds_gateway/static/js/file-list.js | Implemented selection mode toggle, checkbox state management, and row click handling |
| gateway/sds_gateway/static/js/components.js | Updated row click handler to ignore clicks on selection checkboxes |
| gateway/sds_gateway/static/js/actions/QuickAddToDatasetManager.js | New manager class handling modal interactions, dataset loading, and API calls for single/multi capture addition |
| gateway/sds_gateway/static/css/file-list.css | Added CSS rules to show/hide selection column based on selection mode state |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lucaspar
requested changes
Feb 23, 2026
LLKruczek
force-pushed
the
lk/quick-add-capture
branch
from
8c30df5 to
31278c8
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Changes to be committed: modified: gateway/sds_gateway/static/css/file-list.css new file: gateway/sds_gateway/static/js/actions/QuickAddToDatasetManager.js modified: gateway/sds_gateway/static/js/components.js modified: gateway/sds_gateway/static/js/file-list.js modified: gateway/sds_gateway/templates/users/file_list.html modified: gateway/sds_gateway/templates/users/partials/captures_page_table.html new file: gateway/sds_gateway/templates/users/partials/quick_add_to_dataset_modal.html modified: gateway/sds_gateway/users/urls.py modified: gateway/sds_gateway/users/views.py Changes to be committed: modified: gateway/sds_gateway/static/css/file-list.css new file: gateway/sds_gateway/static/js/actions/QuickAddToDatasetManager.js modified: gateway/sds_gateway/static/js/components.js modified: gateway/sds_gateway/static/js/file-list.js modified: gateway/sds_gateway/templates/users/file_list.html modified: gateway/sds_gateway/templates/users/partials/captures_page_table.html new file: gateway/sds_gateway/templates/users/partials/quick_add_to_dataset_modal.html modified: gateway/sds_gateway/users/urls.py modified: gateway/sds_gateway/users/views.py
LLKruczek
force-pushed
the
lk/quick-add-capture
branch
from
dda44b5 to
8722cac
Compare
lucaspar
reviewed
Apr 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.