-
Notifications
You must be signed in to change notification settings - Fork 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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
for granted sso command, add storing parameters in input config https://github.com/common-fate/granted/issues/650 #651
for granted sso command, add storing parameters in input config https://github.com/common-fate/granted/issues/650 #651
Conversation
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.
Thanks very much for this @mikesarver, I am very keen to improve the UX with profile generation and avoiding having to pass all those extra flags. I've requested changes on a couple of smaller nits, as well as some thinking around how we might make this useable for folks who have multiple AWS SSO instances they work with.
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.
Tested this locally and it's working great! Thanks so much @mikesarver 🙌
What changed?
For
granted sso generate/populate
commands add support to persist preferences in .granted/configWhy?
issues/650
Persists
granted sso
preferences in~/.granted/config
. This allows storing preferences in granted config across commands. This allows persisting the data once in config and used across when running commands to sync profiles.How did you test it?
Tested manually.
Potential risks
Is patch release candidate?
Link to relevant docs PRs