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 updates how the "Suggest Security Event over Incident" feature is presented and configured. The main change is to the default value for this feature in the database, and a refactor of the UI alert in the incident report submission form for clarity and better placement.

@aaronherman aaronherman added the enhancement New feature or request label Aug 19, 2025
@aaronherman aaronherman requested a review from Copilot August 19, 2025 16:43
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 PR improves the presentation and default configuration of the "Suggest Security Event over Incident" feature. The key changes include moving the security event suggestion alert from the form submission area to the top of the form for better visibility, and updating the database migration to default this feature to disabled rather than enabled.

  • Relocated the security event suggestion alert from the bottom action area to the top introductory section
  • Changed the default database value for suggest_security_event_over_incident from true to false

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
ReportSubmissionCard.vue Moved the security event suggestion alert from form actions to the top introduction area for better user visibility
2025-08-06_f2bce475e71b.py Updated database migration to set default value for security event suggestion feature to false instead of true

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@aaronherman aaronherman merged commit 11a85ec into main Aug 19, 2025
15 checks passed
@aaronherman aaronherman deleted the fix-suggest-sec-events-modifications branch August 19, 2025 21:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants