Fixes #39532 - All hosts - add action - bulk add capsule#616
Conversation
MariaAga
left a comment
MariaAga
left a comment
There was a problem hiding this comment.
initial review:
Capsule is for downstream, and gets translated from "Proxy" please only use proxy here.
Going to do some tests now, and then a more in depth review once it gets updated
| }; | ||
|
|
||
| const descriptionText = selectAllHostsMode ? ( | ||
| <> |
There was a problem hiding this comment.
FormattedMessage might be helpful here
There was a problem hiding this comment.
I was wondering what will be the best approach, but I decided to not use FormattedMessage so we don't rely on external libs.
There was a problem hiding this comment.
we use react-intl anyway for out translations
There was a problem hiding this comment.
Would you prefer to have it in FormattedMessage or I can leave it like this?
There was a problem hiding this comment.
I think you can leave it as is. was mostly sharing info
There was a problem hiding this comment.
Success message doesn't include host count (inconsistent with core bulk actions)
There was a problem hiding this comment.
I tried changing org and location but it does not show count either
|
Please have a look at this, too: #615 |
| handleModalClose(); | ||
| }; | ||
|
|
||
| const handleError = () => { |
There was a problem hiding this comment.
Would it be possible to re-use the error handling from theforeman/foreman#11045 ?
There was a problem hiding this comment.
@sbernhard Thanks for the feedback. It is actually solving different issues. In my case, I am using handleError for closing the modal and setting state. For handleSuccess I am using for refreshing the page and closing modal.
The foreman PR is solving parsing error/sucess messages from the server response and displaying Toast by using addToast. In my case, I am using directly success/errorToast built in APIActions so it's actually different codes for different purpose.
There was a problem hiding this comment.
I also added my comments to the PR
3 participants
Users can assign an OpenSCAP Proxy to a host. This can be done individually for a host via the host edit form, but also in bulk via the legacy host overview UI (see screenshot). The new UI currently doesn't support this.
Option can be found on AllHosts page -> select hosts -> kebab menu -> Change association -> OpenSCAP Capsule