Skip to content

Fixes #39494 - Fix create report button is enabled when state is 'running'#1041

Merged
MariaAga merged 2 commits into
theforeman:masterfrom
Lukshio:createReport
Jul 14, 2026
Merged

Fixes #39494 - Fix create report button is enabled when state is 'running'#1041
MariaAga merged 2 commits into
theforeman:masterfrom
Lukshio:createReport

Conversation

@Lukshio

@Lukshio Lukshio commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Bug where UI did not check for state 'running' -> button enabled instead of disabled

@MariaAga

MariaAga commented Jul 9, 2026

Copy link
Copy Markdown
Member

@Lukshio

Lukshio commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

it seems like it will clash with https://github.com/theforeman/foreman_remote_execution/pull/1030/changes#diff-753af1c76d3524442c12c00b42c011b9170e139aadccb65a894cdd54904a0795 @pondrejk

It should be ok, the issue is the same, so this will fix it, and the #1030 PR should be fine

@MariaAga

Copy link
Copy Markdown
Member

Could you update your pr? the other one was merged

@Lukshio

Lukshio commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

@MariaAga I also replaced test with RTL store helpers in new commit

@Lukshio

Lukshio commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

Linter issues unrelated. Redmine dont like second commit

@MariaAga MariaAga left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

code looks good, ran the JS tests locally and they pass, and tested with packit that the button is disabled and enabled when needed.

@MariaAga MariaAga merged commit 709bb12 into theforeman:master Jul 14, 2026
14 of 17 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