8000 Adds description as a read-only attribute of WorkbookItem by jorgeFons · Pull Request #533 · tableau/server-client-python · GitHub
[go: up one dir, main page]

Skip to content

Adds description as a read-only attribute of WorkbookItem #533

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 1 commit into from
Dec 4, 2019

Conversation

jorgeFons
Copy link
Contributor

This exposes the description field that is returned by the API as a read-only attribute on the WorkbookItem model.

@jorgeFons jorgeFons changed the base branch from master to development November 26, 2019 17:18
8000
@jorgeFons jorgeFons force-pushed the add_workbook_description branch from a3d9a94 to dd14cb8 Compare November 26, 2019 17:20
@jorgeFons
Copy link
Contributor Author

@shinchris I can't request review for this. I might've misread the contributing guidelines though, what should I do to unblock?

@shinchris
Copy link
Contributor

Thanks for making the change! 😃 🚀
I'm not sure why you aren't able to request reviewers, but it looks good to merge in.

In order to proceed with the merge, could you read and sign our CLA?
http://tableau.github.io/contributing.html

@jorgeFons
Copy link
Contributor Author

Thank you @shinchris.
Just mailed the CLA.

@james-baker
Copy link

CLA received and approved.

@shinchris shinchris merged commit 308705d into tableau:development Dec 4, 2019
shinchris pushed a commit that referenced this pull request Feb 21, 2020
* Added a way to handle non-xml errors (#515)
* Added Webhooks endpoints for create, delete, get, list, and test (#523, #532)
* Added delete method in the tasks endpoint (#524)
* Added description attribute to WorkbookItem (#533)
* Added support for materializeViews as schedule and task types (#542)
* Added warnings to schedules (#550, #551)
* Added ability to update parent_id attribute of projects (#560, #567)
* Improved filename behavior for download endpoints (#517)
* Improved logging (#508)
* Fixed runtime error in permissions endpoint (#513)
* Fixed move_workbook_sites sample (#503)
* Fixed project permissions endpoints (#527)
* Fixed login.py sample to accept site name (#549)
shinchris pushed a commit that referenced this pull request Feb 21, 2020
Merging v0.10 changes from development to master

* Added a way to handle non-xml errors (#515)
* Added Webhooks endpoints for create, delete, get, list, and test (#523, #532)
* Added delete method in the tasks endpoint (#524)
* Added description attribute to WorkbookItem (#533)
* Added support for materializeViews as schedule and task types (#542)
* Added warnings to schedules (#550, #551)
* Added ability to update parent_id attribute of projects (#560, #567)
* Improved filename behavior for download endpoints (#517)
* Improved logging (#508)
* Fixed runtime error in permissions endpoint (#513)
* Fixed move_workbook_sites sample (#503)
* Fixed project permissions endpoints (#527)
* Fixed login.py sample to accept site name (#549)
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.

4 participants
0