8000 Development by graysonarts · Pull Request #107 · tableau/document-api-python · GitHub
[go: up one dir, main page]

Skip to content

Development #107

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 19 commits into from
Nov 1, 2016
Merged

Development #107

merged 19 commits into from
Nov 1, 2016

Conversation

graysonarts
Copy link
Contributor

Getting Development updated with master's history. It looks big but it's mostly just rebasing to fix up the history. This one needs to be approved before #106

t8y8 and others added 19 commits October 7, 2016 10:47
* Scrub some strings in the test file so they don't include internal references
* Updated some of the assets to be consistent in casing
* Create CONTRIBUTORS.md

* Adding Charley Peng who contributed doc fixes

* Update CONTRIBUTORS.md
When Cross Database Joins were introduced in 10 all connections became "federated" by default, even if they weren't actually joined to anything else. 

In the file format that means they get represented as named-connection/connection elements. Expressed under one top-level connection element with a class of 'federated'.

Except for 'sqlproxy' connections (Published Data Sources) -- they stay in the old connection style as a top level connection element.

We need to, when in a 10.0 or greater workbook, get all federated connections (named-connection) plus go back and find any sqlproxy connections as well.
* Add the ability to clear repository location on the datasource for retargetting
* fixing a pep8 issue that was missed
* Adding None check for repository-location in case it doesn't exist
* Updating link in readme

We renamed Examples to samples, but forgot to update the readme.

* Adding #80 to changelog
Issue #96: Add port support

We now support reading and setting the port attribute on connections. If you set the port attribute to None it will remove it entirely. Small change to tests so that they don't have side effects between test cases.
* Add caption support for datasources

* fix a pep8 issue E303

* Add ability to remove the caption using del
* Jekyll Dependencies
* First draft of docs -- we'll improve once skeleton is in place
* Does not deploy yet, need to do that in github settings page
Added a few periods, etc.
@graysonarts graysonarts merged commit 07f8487 into tableau:development Nov 1, 2016
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.

5 participants
0