Skip to content

Fixes #39425 - Update RoleSwitcherError from pf3 to pf5#793

Merged
kmalyjur merged 1 commit into
theforeman:masterfrom
pondrejk:switcher-error
Jun 22, 2026
Merged

Fixes #39425 - Update RoleSwitcherError from pf3 to pf5#793
kmalyjur merged 1 commit into
theforeman:masterfrom
pondrejk:switcher-error

Conversation

@pondrejk

Copy link
Copy Markdown
Contributor

part of pf3 to pf5 conversion, supplements #792

@kmalyjur kmalyjur left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I get this error. Do you see it, too?
It seems like Col doesn't exist in pf5.

Image

@pondrejk

Copy link
Copy Markdown
Contributor Author

It seems like Col doesn't exist in pf5.

Uh, true! Amended, thanks :)

error && error.errorMsg ? (
<Col sm={12}>
<Alert type="error">
<Alert ouiaId="ansible-roles-switcher-error" type="error">

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the pf5 it's variant="danger"

<Col sm={12}>
<Alert type="error">
<Alert ouiaId="ansible-roles-switcher-error" type="error">
<ErrorMsg error={error} />

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that instead of ErrorMsg, the Alert's title could be used like this:
<Alert variant="danger" title="Danger alert title" ouiaId="DangerAlert" />

@pondrejk

Copy link
Copy Markdown
Contributor Author

@kmalyjur thank you, I think I've applied all the suggestions

@kmalyjur kmalyjur left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Image

@kmalyjur kmalyjur merged commit c19166c into theforeman:master Jun 22, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants