Added vulnerabilities inventory that affect to an agent #3069
Conversation
| const filters = filtersToObject(this.state.filters); | ||
| const filter = { | ||
| ...filters, | ||
| limit: '15' |
There was a problem hiding this comment.
Because is just the default value. The page size value can be selected within the table component.
There was a problem hiding this comment.
But if the page size changes, the limit variable continues on 15, no?
| this.showToast('success', 'Your download should begin automatically...', 3000); | ||
| await exportCsv( | ||
| '/vulnerability/' + this.props.agent.id, | ||
| [{}, |
There was a problem hiding this comment.
My bad, I forgot to deleted. I'll fix it right away
| return ( | ||
| <Fragment> | ||
| <EuiAccordion | ||
| id={id === undefined ? Math.random().toString() : `${id}_details`} |
There was a problem hiding this comment.
I am not sure about this. When the id variable will be undefined? The EuiAccordion needs an id prop?
| initialColumns={[ | ||
| 'icon', | ||
| 'timestamp', | ||
| // 'rule.mitre.id', |
There was a problem hiding this comment.
If this field is not necessary, you could remove it.
| componentWillUnmount() { | ||
| } |
|
Testing:
|
Is there a chance someone wants a report showing there are no vulnerabilities? Should we deny that possibility? |
The |
Fair enough, disabling the button when there are no vulnerabilities is the quickest solution to that issue. |
frankeros
deleted the
feature/vulnerabilities-view-that-affect-each-agent
branch
frankeros
changed the title
Hi team!
This adds the new vulnerabilities per agent view and it's details flyout.
Closes #3011