-
-
Notifications
You must be signed in to change notification settings - Fork 167
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
Fix affected projects tab not being updated when switching between vulnerability aliases #509
Fix affected projects tab not being updated when switching between vulnerability aliases #509
Conversation
Hello @valentijnscholten, would you please check if this actually fixes the problem you described in #481? |
The response in affected projects from the backend will be different for an alias, that's the reason the tab must be refreshed. |
@sephiroth-j Sorry for the late reply, needed some time to get DT going again locally and be able to build the PR. |
Oh sorry, there were some errors running |
fixes DependencyTrack#481 Signed-off-by: Ronny Perinke <[email protected]>
c652559
to
7fc4669
Compare
Hi @valentijnscholten, I have rebased the bugfix branch to 4.12.x and updated the target branch of the PR accordingly. |
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.
thanks, it still works :-D
@sephiroth-j Can you sign your commits? |
@valentijnscholten my commit is signed. I think you just need to re-run the DCO check. |
@sephiroth-j I can't. I think it'll run if close->reopen the PR. |
I have manually passed the DCO check since @sephiroth-j's commit is indeed signed off. |
Description
The affected projects tab (and table) is not updated correctly when switching between aliases.
Addressed Issue
fixes #481
Additional Details
I'm leaving this as a draft for now because there is also a backend issue that should be fixed first (see #481 (comment)).
Checklist
This PR introduces new or alters existing behavior, and I have updated the documentation accordingly