Skip to content
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

Feature: add --org option to all relevant CLI commands. #1136

Merged
merged 3 commits into from Jun 20, 2016
Merged

Feature: add --org option to all relevant CLI commands. #1136

merged 3 commits into from Jun 20, 2016

Conversation

adamlwgriffiths
Copy link
Contributor

--org is the organization slug, not the name.
Allows the management of users, datasources and groups
with respect to organisations.
All commands default to 'default' slug, or None where
relevant, which means the commands will still work
as they did before without any changes.

--org is the organization slug, not the name.
Allows the management of users, datasources and groups
with respect to organisations.
All commands default to 'default' slug, or None where
relevant, which means the commands will still work
as they did before without any changes.
@@ -7,8 +7,7 @@
manager = Manager(help="Users management commands. This commands assume single organization operation.")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You will need to update this :)

Move organization to the end of the parameter
list to prevent breaking any existing code.
Provide a default for all organization parameters
to allow people to call the CLI functions without
breaking existing code.
Add missing organization parameter to `list` in
`users.py`.
Add "(leave blank for 'default')." or
"(leave blank for all organizations)" to organization options.
Remove the text "This commands assume single organization operation."
from the `users` and `groups` managers help text.
@adamlwgriffiths
Copy link
Contributor Author

I've updated that text for both users and groups managers.
I added a missing organization parameter to the users.list function.
I added appropriate defaults to the organization parameters to allow calling the functions manually without having to specify the organization.
I also moved the organization parameter to the end of each function it was added to, to avoid breaking any existing code.

@arikfr arikfr changed the title Add --org option to all relevant CLI commands. Feature: add --org option to all relevant CLI commands. Jun 20, 2016
@arikfr arikfr merged commit d5e5b24 into getredash:master Jun 20, 2016
@arikfr
Copy link
Member

arikfr commented Jun 20, 2016

Merged. Thanks!

I'll be happy to hear on how you're using this. There is no email in your profile, so you're welcome to reach out (arik at redash.io).

simo7 pushed a commit to pubnative/redash that referenced this pull request Sep 22, 2016
Feature: add --org option to all relevant CLI commands.
dairyo pushed a commit to KiiCorp/redash that referenced this pull request Mar 1, 2019
Feature: add --org option to all relevant CLI commands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants