-
Notifications
You must be signed in to change notification settings - Fork 5
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
Show an icon if the organization has credentials setup or not #850
Conversation
- Add getting credentials to getting organizations action - Show icons in the table view if the organization has credentials setup or not - Clean up code
This new column would only make sense on AWS and Azure, not on KVM. I don't see any condition for this so far. While I like that you are using an icon for this column, I don't find the ✖️ icon ideal. We use it for closing modals. Can we find an alternative? What do others think? |
I actually didn't know that, I'll make the change now. Thank you 👍 |
…and hide the column if the provider is 'kvm'
How about only using the CHECK icon and just leaving it blank for when there are no credentials? |
Looks pretty good. What do you think, @marians ? |
Like it! |
…m/giantswarm/happa into Add-credential-check-to-orgs-list
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.
LGTM!
Let's merge this? |
Fixes giantswarm/giantswarm#6630
What it looks like