- Notifications
You must be signed in to change notification settings - Fork923
chore: remove org context switcher in the cli#13674
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
d0e9124 to2917474Comparecli/root.go Outdated
| // Otherwise, we can assume a default value. | ||
| Required: false, | ||
| Flag: "org", | ||
| FlagShorthand: "z", |
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.
-z is a really weird choice here imo. I know-o is used, but what about-O? just doesn't feel like there is very clear connection.
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.
I was thinking organiZation lol. I can switch to-O
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.
thank you 😄
5bf46f3 intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
What this does
Removes the cli organization switcher. All commands that require an organization context default to an organization if the user is only in 1 org. Multi org users require setting
CODER_ORGANIZATION=<org>or--org=<organization>(-O=<organization>shorthand)Future Work
Ideally we could remove the organization context required from many of the template related commands if the template name is unambiguous. At present, the organization ID is required for the fetching template API.