8000 Merge in v0.7 from development. by graysonarts · Pull Request #316 · tableau/server-client-python · GitHub
[go: up one dir, main page]

Skip to content

Merge in v0.7 from development. #316

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 18 commits into from
Jul 11, 2018
Merged

Merge in v0.7 from development. #316

merged 18 commits into from
Jul 11, 2018

Conversation

graysonarts
Copy link
Contributor

This required a bit of work because of the as_job getting accidentally checked into master.

Russell Hay and others added 18 commits January 31, 2018 14:24
* Implement view filters on the populate* request options
* Adding export sample/tool

* Removing unused import

* Addressing code review feedback

* Missed one description
* Change to correct parent project id tag name

* Change to correct parent project id tag name
* add export_wb sample for 'fullpdf'

* Addressing code review feedback from Lee

* Fixing the dumb thing that I did where I was getting the view, then throwing it away and getting it again...

* pep8 error
…entries.

Tested manually, and a simple regression test added to the suite.
* addressing bug #281: update datasource connection not updating server port properly
…284)

* adding ability to rename workbook using workbooks.update()

* updating test
8000
* Adding update method for groups
* Add some docs
* Add test for update group function
Taking @marianotn 's PR and updating it to retain backwards compatibility. Pre-implemented for datasources as well, though those don't support this server-side yet, I've set that version to 99.99. When support lands we can update it to the version it's introduced in.
* initial checkin for get background jobs

* pep8 fixes

* addressed code review feedback
* added cancel job
* fixing whitespace issues
* addressing small nits in the sample and the docs
* Adding chnagelog for 0.7

* Adding Sergey to contributors

* Adding async to the changelog
Merging things since as_job was added in master by accident
@graysonarts graysonarts requested a review from shinchris July 10, 2018 21:38
@t8y8
Copy link
Collaborator
t8y8 commented Jul 11, 2018

Whoo-wee, that's a big PR.

We've worked on a lot!

Copy link
Collaborator
@t8y8 t8y8 left a comment

Choose a reason for hiding this comment

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

🚀

@graysonarts graysonarts merged commit b7b544e into tableau:master Jul 11, 2018
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.

6 participants
0