-
Notifications
You must be signed in to change notification settings - Fork 53
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
feat(vault): Add incomplete setup warning to machine list #4503
feat(vault): Add incomplete setup warning to machine list #4503
Conversation
Demo starting at https://maas-ui-4503.demos.haus |
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.
Appreciate small size of the PRs, makes them much easier to review!
@ndv99 QA+1 Integrate with Vault is labeled as a region correctly playing nicely with VoiceOver. |
Awesome, cheers! |
Done
QA
MAAS deployment
To run this branch you will need access to one of the following MAAS deployments:
Running the branch
You can run this branch by:
QA steps
Fixes
Fixes https://github.com/canonical/app-tribe/issues/1407
Screenshots
Not all controllers configured
Secrets not migrated