-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added test plans for Advisory and Vulnerability explorers. #11
base: main
Are you sure you want to change the base?
Added test plans for Advisory and Vulnerability explorers. #11
Conversation
After Pavel's feedback, I've updated the MR with a better use of the keywords as they are defined in the docs. |
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.
Please take a look and let me know WDYT?
When User navigates to the Vulnerabilities Explorer page | ||
Then The ID and Title of each vulnerability is visible on each row | ||
And "Show more" button is visible for vulnerabilities with a long title | ||
And Advisory metadata (CVSS, Date published, Related documents) is visible on each row |
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.
This should be updated Vulnerability metadata CVE name, CVSS score and CVE description
@mrrajan I was a bit confused about what the difference between an overview and an explorer was. Now it's clear. Thanks for the suggestions, I've made the changes. |
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.
I missed the unresolved conversations. Revoking my approval.
d664a61
to
a7cae26
Compare
@mrrajan Okay, fixed the last remaining things that you pointed out. |
Co-authored-by: Rajan Ravi <86094767+mrrajan@users.noreply.github.com>
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.
LGTM
No description provided.