Skip to content

Conversation

@rehsfl
Copy link
Contributor

@rehsfl rehsfl commented Oct 20, 2025

Changes proposed in this pull request:

  • Move "new assessment" above history in VulnModal

Status

  • READY
  • HOLD
  • WIP (Work-In-Progress)

How to verify this change

Open the web dashboard and pick a vulnerability from the Vulnerabilities tab, observe that the New assessment section moved to be at the top of the assessments history

Pull Request Checklist

Please review and check all that apply before submitting your PR:

  • The code compiles and passes all tests
  • All new and existing tests are passing
  • Documentation has been updated (if applicable)
  • Code follows project style guidelines
  • No sensitive information is included
  • Linked relevant issues (if any)
  • Added necessary reviewers

@rehsfl rehsfl changed the base branch from main to staging October 20, 2025 19:08
@ValentinBoudevin
Copy link
Contributor

LGTM but add a description to your commit

Moving the new assessment form to the top of assessments history
because the history is shown in most recent order and the form
was below the earliest assessment
@rehsfl rehsfl force-pushed the add-assessment-top branch from 6c6d744 to 71b1395 Compare October 23, 2025 15:13
Copy link
Contributor

@ValentinBoudevin ValentinBoudevin left a comment

Choose a reason for hiding this comment

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

LGTM

@ValentinBoudevin ValentinBoudevin merged commit e40f630 into savoirfairelinux:staging Oct 23, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants