- Notifications
You must be signed in to change notification settings - Fork13
Optionally match on username#90
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Thanks for the PR,@timle8n1 san! I have been busy and sorry for the late review. I too think changing the color according to the username is a good idea. On the other hand, I was worried that if it becomes so multifunctional, there might be cases where you want to change the conditions for the label foreground and background. Do you know of any UI management tools that allow such flexible settings, and what kind of setting strategies they use? Note that the above concern is just a precaution, and I basically agree with PR's policy! |
Hey thanks for the reply. And thanks for the tool! My company recently switched from native AWS cross account roles that allow you to set the color of each role that is remembered. They suggested this tool to replace the color switching that many of us were using. However I ran into the issue that I login to the same AWS account with two different roles (read only and admin effectively). This PR is just to scratch that itch, I’m not too sure what other tools might use. I’m running personally on my fork of this and it’s working fine but totally understand if you don’t want to merge it. Whatever you decide, thanks for the tool - it works great! |
Thank you very much! I see, you originally used native cross account roles. I used it myself, so I understand. Thank you for using this! |
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.
@timle8n1 Reviewed PR.
I thought about comparing it to the existing Switch Role as well, but I think we should use roleName instead of userName.
Since userName is different for each user, roleName is more appropriate when the team is managing roles.
I will try to fix this PR when I have time, but of course you are welcome to fix it on your end!
@timle8n1 I have reconsidered. It is difficult to follow AWS specifications, so I am considering allowing the registration of regular expression patterns for UserName. I would like to keep your contribution, so I will start from this PR. |
Summary
Matches on username optional like region
Motivation
When accessing environments we have various roles that show up in the form of the username with SSO - this allows for changing the color based on the username/role - so edit contexts can be a different color than just read contexts