Skip to content

Remove Status dropdown in All Vulns table (CRASM-1090)#847

Merged
rapidray12 merged 2 commits intodevelopfrom
Disable-Vuln-Status-Dropdown-CRASM-1090
Apr 9, 2025
Merged

Remove Status dropdown in All Vulns table (CRASM-1090)#847
rapidray12 merged 2 commits intodevelopfrom
Disable-Vuln-Status-Dropdown-CRASM-1090

Conversation

@hawkishpolicy
Copy link
Contributor

  • Remove the Status dropdown from the All Vulnerabilities table.

🗣 Description

  • Per CRASM-1090, commmented out the status update dropdown in All Vulns table.
  • Replaced with static text to indicate the vuln status.
  • Will be re-enabled at a future date.

💭 Motivation and context

  • Closes CRASM-1090

🧪 Testing

  • tested locally

📷 Screenshots (if appropriate)

Screenshot 2025-04-07 at 3 49 52 PM Screenshot 2025-04-07 at 3 49 37 PM

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All future TODOs are captured in issues, which are referenced
    in code comments.
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All new and existing tests pass.

✅ Pre-merge checklist

  • Revert dependencies to default branches.
  • Finalize version.

✅ Post-merge checklist

  • Create a release.

- Per CRASM-1090, commmented out the status update dropdown in All Vulns table.
- Replaced with static text to indicate the vuln status.
- Will be re-enabled at a future date.
Copy link
Collaborator

@rapidray12 rapidray12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed in a group meeting

@rapidray12 rapidray12 merged commit 5b26575 into develop Apr 9, 2025
15 of 16 checks passed
@rapidray12 rapidray12 deleted the Disable-Vuln-Status-Dropdown-CRASM-1090 branch April 9, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants