8000 Push code for 0.25 with custom views by jacalata · Pull Request #1206 · tableau/server-client-python · GitHub
[go: up one dir, main page]

Skip to content

Push code for 0.25 with custom views #1206

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 29 commits into from
Mar 16, 2023
Merged

Push code for 0.25 with custom views #1206

merged 29 commits into from
Mar 16, 2023

Conversation

jacalata
Copy link
Contributor
@jacalata jacalata commented Mar 9, 2023

add query-tagging attribute to connection (#1202)
catch exceptions from ServerInfo (#1204)
Fix bug in update-datasources before 3.15 (#1203)
Implement custom view objects (#1195)
⬆️ Allow more recent version of packaging (#1196)
fix behavior when url has no protocol
add repr for more models (#1191)
cleaner imports (#1193)

jacalata and others added 29 commits September 22, 2022 15:50
* ssl-verify is an option, not a header
* Allow injection of session_factory to allow use of a custom session
Pylint with "errors only" isn't 100% accurate, but it found a few problems
that should be fixed.
* fix behavior when url has no protocol
* smoke test for pypi
* Add permission control for Data Roles and Metrics (#1)
* Add functions to control permissions
* Add new resource types
bring over fix from test repo
* add option to pass specific datasources
* mypy no-implicit-optional
Allow publishing using connection credentials on ConnectionItem class without
ConnectionCredentials instance, as documented

Accept empty string for username or password in connection credentials

Avoid Tableau Server internal server error when publishing with empty
connection list by setting connections to None
* fix revision references where xml returned does not match docs
This should fix (1) from #1139 (comment), by preventing us from sending an empty connections element if the list of connections is empty.
remove warnings

fix behavior when url has no protocol
* make imports absolute

was: "from ..models import x"
now: "from tableauserverclient.models import x"

* IntelliJ optimize imports

also some manual work


this will make it easier to split them into properly independent modules
* add repr to several models
* test all model classes for __repr__

add a dumb listing of all existing model classes, and a test that iterates over the whole list. Uses subtests so we get the whole list.

Test is skipped because we know they don't all have it yet. Will update to skip known not-implemented models instead.
* create custom views item and endpoint with get/update/delete methods
Also added custom view operations to the workbook sample
require project id for updating datasource before 3.15
for #1072
* catch all exceptions from ServerInfoEndpoint
remove use of warnings
add check before making http request
* add query-tagging attribute to connection
* add explanation for why it doesn't work on hyper
@jacalata jacalata merged commit e4fbe41 into master Mar 16, 2023
@jacalata jacalata deleted the development branch March 16, 2023 18:05
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.

8 participants
0