New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #36089 - Job wizard default to no hosts selected #784
Conversation
Need to change the message + logic to allow 0 hosts selected but not allow no host selection |
9de246f
to
53ed208
Compare
Redone the pr to treat the hosts field as a required field |
c5638e9
to
5d6437b
Compare
Once I fill in required inputs, it allows me to go to the review step and kick the job off even though no hosts are selected. Is that intentional? |
5d6437b
to
4530f5a
Compare
missed condition check in the last step, added it. review should be disabled if there are no hosts/search query |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works well
Thank you @MariaAga ! |
using the magic of1=0
like ruby does atif displayed_search_query.blank? Host.where('1 = 0')
Checking if any of the host selections are selected after a user clicks the host input, if not it will show an error