8000 default permissions by jacalata · Pull Request #1054 · tableau/server-client-python · GitHub
[go: up one dir, main page]

Skip to content

default permissions #1054

10000
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

Merged
merged 7 commits into from
Jun 4, 2022
Merged

default permissions #1054

merged 7 commits into from
Jun 4, 2022

Conversation

jacalata
Copy link
Contributor
@jacalata jacalata commented Jun 4, 2022

Refactored the repeated use of strings for type names into a class. Added str and repr methods to a bunch of resources and added sample code, to properly examine the code.

@jacalata
Copy link
Contributor Author
jacalata commented Jun 4, 2022

Example of output from new repr
INFO:root:{'type': '_default_datasource_permissions', 'value': [<PermissionsRule grantee=, capabilities={'Read': 'Allow', 'Write': 'Allow', 'Connect': 'Allow'}>]}

INFO:tableauserverclient.server.endpoint.default_permissions_endpoint:{'content_type': 'datasource', 'permissions': [<PermissionsRule grantee=, capabilities={'Read': 'Allow', 'Write': 'Allow', 'Connect': 'Allow'}>]}

@jacalata jacalata merged commit f41aa09 into development Jun 4, 2022
@jacalata jacalata deleted the jac/amar-kumar-yadav-1044 branch June 4, 2022 03:26
jacalata added a commit that referenced this pull request Jun 6, 2022
* added new permission populate methods
* refactor Resource Types and add sample code accessing default permissions
co-authored-by: Amar Yadav <AmarKumar.Yadav@genmills.com>
jacalata added a commit that referenced this pull request Sep 20, 2022
* added new permission populate methods
* refactor Resource Types and add sample code accessing default permissions
co-authored-by: Amar Yadav <AmarKumar.Yadav@genmills.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0