Skip to content

fix(iop): add missing remed modal to pathway props#1232

Merged
jeremylenz merged 1 commit into
theforeman:developfrom
Fewwy:add-missing-remediations-modal-props
Jul 15, 2026
Merged

fix(iop): add missing remed modal to pathway props#1232
jeremylenz merged 1 commit into
theforeman:developfrom
Fewwy:add-missing-remediations-modal-props

Conversation

@Fewwy

@Fewwy Fewwy commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

What are the changes introduced in this pull request?

Add missing remediations modal props so the pathway details systems table won't crash

Considerations taken when implementing this change?

What are the testing steps for this pull request?

Test it in iop environment using https://github.com/RedHatInsights/insights-advisor-frontend/tree/foreman-pathways-dev-v2 branch and iop-dev repository instructions

@sourcery-ai sourcery-ai Bot 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.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Comment thread webpack/IopRecommendationDetails/IopRecommendationDetails.js

@jeremylenz jeremylenz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Lint errors are unrelated (caused by theforeman/foreman#11008)

@jeremylenz
jeremylenz merged commit 41968c9 into theforeman:develop Jul 15, 2026
16 of 18 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