Affected packages #2763
Merged
Affected packages #2763
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
janvhs
added
wip
javascript
|
Would love to get your feedback, so I can copy paste it for the affected hosts section |
dottorblaster
requested changes
Jul 10, 2024
This patch adds the affected packages to the advisory errata endpoint and adds an openAPI spec for the changes.
This patch adds unit test cases and updates the existing cases to include the new data available from the affected packages discovery.
This patch updates the AdvisoryDetails component to display the affected packages.
…ckages This patch updates and adds the frontend test and story to include the affected packages data.
This patch adds the affected systems from the discovery result to the errata details response. Therefore, this patch contains an update to the openAPI specification. Furthermore, it adds new and updates existing tests to include the changes.
This patch rewords the openAPI specification for the affected packages. I changed the words, to clarify the difference between a package version and a release.
This patch displays the systems affected by a advisory errata to the user. It exposes the new backend API in the advisory errata page and updates the relevant tests. This change required an update to the factories. Furthermore, I added a section for showcasing the "empty data" state in the advisory details page.
janvhs
force-pushed
the
affected-packages
branch
from
f40cd4f to
ceebf41
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Display the affected packages and hosts (WIP) in the advisory details page.
How was this tested?
Elixir unit tests, Jest tests, Storybook