8000 Document the `metadata.query` method by vogelsgesang · Pull Request #896 · tableau/server-client-python · GitHub
[go: up one dir, main page]
Skip to content

Document the metadata.query method #896

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 23, 2021
Merged

Document the metadata.query method #896

merged 1 commit into from
Sep 23, 2021

Conversation

vogelsgesang
Copy link
Collaborator

No description provided.

t8y8
t8y8 suggested changes Sep 20, 2021
View reviewed changes
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.

Minor changes, but thanks for getting it started!

Docs folks should probably take a look too.

@vogelsgesang vogelsgesang force-pushed the gh-pages-metadata-query branch from a78e5dc to 0d8befd Compare September 21, 2021 10:21
@vogelsgesang
Copy link
Collaborator Author

@t8y8 I addressed your issues in the new commit. Please double-check and let me know if there is anything else I should change

t8y8
t8y8 approved these changes Sep 21, 2021
View reviewed changes
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.

🚀

@bcantoni bcantoni merged commit fa31210 into tableau:gh-pages Sep 23, 2021
@bcantoni
Copy link
Contributor

Thanks @vogelsgesang for the addition; merged.

@vogelsgesang vogelsgesang deleted the gh-pages-metadata-query branch September 23, 2021 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t8y8 t8y8 t8y8 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vogelsgesang @bcantoni @t8y8
0