-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Fix TypeError when running a command on a host in a smart inventory #11768
Conversation
4c8a8ab
to
06dc72c
Compare
06dc72c
to
b82d70d
Compare
Fix TypeError when running a command on a host in a smart inventory See: ansible#11611
b82d70d
to
92eb26a
Compare
@@ -91,6 +99,7 @@ function SmartInventoryHostList({ inventory }) { | |||
adHocItems={selected} | |||
hasListItems={count > 0} | |||
onLaunchLoading={setIsAdHocLaunchLoading} | |||
moduleOptions={moduleOptions} |
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.
👍 Verified that adding this prop resolves the issue. Thanks for defining the PropTypes in AdHocCommands. Nice work!
I triggered the e2e tests. |
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
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.
Great work! Running some E2E testing and will merge after that is completed but its looking good so far
…nsible#11768) Fix TypeError when running a command on a host in a smart inventory See: ansible#11611
…nsible#11768) Fix TypeError when running a command on a host in a smart inventory See: ansible#11611
…nsible#11768) Fix TypeError when running a command on a host in a smart inventory See: ansible#11611
…nsible#11768) Fix TypeError when running a command on a host in a smart inventory See: ansible#11611
Fix TypeError when running a command on a host in a smart inventory
See: #11611