-
Notifications
You must be signed in to change notification settings - Fork 59
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
[AAP-23818] Credentials plugins wizard test button #2327
[AAP-23818] Credentials plugins wizard test button #2327
Conversation
3c7c6ca
to
f65e838
Compare
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.
Looks good so far. I would move the toast alert into the credentials plugin modal rather than passing it down and remove the unnecessary else
statement.
frontend/awx/access/credentials/CredentialPlugins/CredentialPlugins.tsx
Outdated
Show resolved
Hide resolved
frontend/awx/access/credentials/CredentialPlugins/hooks/useCredentialPluginsDialog.tsx
Outdated
Show resolved
Hide resolved
frontend/awx/access/credentials/CredentialPlugins/hooks/useCredentialPluginsDialog.tsx
Show resolved
Hide resolved
frontend/awx/access/credentials/CredentialPlugins/hooks/useCredentialPluginsDialog.tsx
Show resolved
Hide resolved
frontend/awx/access/credentials/CredentialPlugins/hooks/useCredentialPluginsDialog.tsx
Show resolved
Hide resolved
frontend/awx/access/credentials/CredentialPlugins/hooks/useCredentialPluginsDialog.tsx
Outdated
Show resolved
Hide resolved
frontend/awx/access/credentials/CredentialPlugins/hooks/useCredentialPluginsDialog.tsx
Outdated
Show resolved
Hide resolved
frontend/awx/access/credentials/CredentialPlugins/hooks/useCredentialPluginsDialog.tsx
Outdated
Show resolved
Hide resolved
028eb65
to
bd92d47
Compare
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.
Should be good to go once the alert toaster is moved into their respective useCredentialPluginsModal and useCredentialTestModal hooks. ⭐
4bbe6f7
to
340a9b3
Compare
340a9b3
to
1a74036
Compare
This PR adds 'test' button to the credentials plugins wizard
Jira issue: https://issues.redhat.com/browse/AAP-23818