This repository was archived by the owner on Sep 3, 2025. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 641
Manage participants from UI #6087
Merged
Merged
Conversation
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
…n_manage_participants
…e_participants_from_ui
…lix/dispatch into manage_participants_from_ui
Contributor
Author
|
@whitdog47 @aaronherman all checks are passing now! can I get a review whenever y'all get a chance? thank you! |
whitdog47
reviewed
Jun 26, 2025
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements participant management functionality for both incidents and cases, allowing users to add and remove participants directly from the UI. The implementation includes confirmation dialogs, API endpoints, and automatic integration with conversation systems like Slack.
- Adds participant management UI with confirmation dialogs and dropdown menus for both incidents and cases
- Implements backend API endpoints for adding/removing participants with automatic conversation integration
- Enhances Slack conversation handling with membership validation and improved error handling
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
src/dispatch/static/dispatch/src/incident/store.js |
Adds state management for remove participant dialog |
src/dispatch/static/dispatch/src/incident/api.js |
Implements API methods for adding/removing incident participants |
src/dispatch/static/dispatch/src/incident/RemoveParticipantDialog.vue |
Creates confirmation dialog component for incident participant removal |
src/dispatch/static/dispatch/src/incident/ParticipantsTab.vue |
Updates UI with participant management controls and filtering |
src/dispatch/static/dispatch/src/case/store.js |
Adds state management for case participant dialog handling |
src/dispatch/static/dispatch/src/case/api.js |
Implements API methods for adding/removing case participants |
src/dispatch/static/dispatch/src/case/RemoveParticipantDialog.vue |
Creates confirmation dialog component for case participant removal |
src/dispatch/static/dispatch/src/case/ParticipantsTab.vue |
Updates case UI with participant management functionality |
src/dispatch/plugins/dispatch_slack/service.py |
Adds membership validation before removing users from Slack channels |
src/dispatch/plugins/dispatch_slack/enums.py |
Adds new Slack error code enum |
src/dispatch/participant/flows.py |
Refactors participant reactivation to support both cases and incidents |
src/dispatch/incident/views.py |
Adds REST endpoints for participant management |
src/dispatch/incident/flows.py |
Enhances participant flows with conversation integration and event logging |
src/dispatch/conversation/flows.py |
Adds event logging for case participant additions |
src/dispatch/case/views.py |
Adds REST endpoints for case participant management |
src/dispatch/case/flows.py |
Enhances case participant removal with conversation integration |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Nathan Yee <nathanmyee@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Nathan Yee <nathanmyee@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Nathan Yee <nathanmyee@gmail.com>
whitdog47
approved these changes
Jul 22, 2025
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
This PR adds the ability to manage (e.g. add or remove) participants from the UI.
Frontend Changes
RemoveParticipantDialog.vuefor both incidents and casesremoveParticipantandaddParticipantmethods to both incident and case APIsBackend Changes
/{case_id}/remove/{email},/{case_id}/add/{email},/{incident_id}/remove/{email}, and/{incident_id}/add/{email}Screenshots

Timeline event for being added to a conversation
Timeline event for being removed from a conversation

New UI to add participants from the dropdown menu and remove participants with the delete icon

Confirmation dialog before removing a participant
