Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

Conversation

@aaronherman
Copy link
Contributor

This pull request refactors the handle_escalation_submission_event function in src/dispatch/plugins/dispatch_slack/case/interactive.py to improve code clarity by renaming imported functions and using the correct one for priority.

@aaronherman aaronherman requested a review from Copilot July 24, 2025 16:16
@aaronherman aaronherman self-assigned this Jul 24, 2025
@aaronherman aaronherman added the bug Something isn't working label Jul 24, 2025
Copy link
Contributor

Copilot AI left a 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 pull request fixes a bug where escalation priority was not persisting correctly in the case escalation workflow. The issue was caused by using the wrong service function to retrieve incident priorities.

  • Renamed imported functions to disambiguate between incident type and priority services
  • Fixed priority retrieval by using the correct priority service function instead of the type service function

@aaronherman aaronherman requested a review from whitdog47 July 24, 2025 16:20
@aaronherman aaronherman merged commit 3ed8efa into main Jul 24, 2025
9 checks passed
@aaronherman aaronherman deleted the fix/case-escalation-priority branch July 24, 2025 22:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants