GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Release 0.3 by graysonarts · Pull Request #78 · tableau/document-api-python

Conversation

@graysonarts

@graysonarts graysonarts commented Aug 31, 2016

Copy link
Copy Markdown
Contributor

No description provided.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
t8y8 and others added 11 commits July 27, 2016 11:20
@t8y8
Basic connection class retargeting (#65)
44ca620
- Allow modification of dbclass attribute
- Only allow updates to a valid (known) dbclass type.

That is, you can change the database type. 
Other attributes (like ssl and all the various auth modes are not yet supported)

Now you can take: mysql-test-db.yourcompany.com, and move it to postgres-prod-db.yourcompany.com and update the dbclass to match!
@t8y8
Bump test coverage in fields: cover datatype and role attributes (#64)
7041c9e
* Bump test coverage up a bit in fields
* Fix pep8 violation, and it fixed a few other things automatically as well
@t8y8
Make sure we are loading the expected filetype (#62)
4d6ff51
* Proper validation that Workbooks load 'twb(x)' and Datasources load 'tds(x)' files only
* Small cleanup, move the try inside the with

* Add some tests to make sure I don't break anything and cleanup a few small bits
Russell Hay
address lack of coverage from #63 (#67)
e0d1eb5
Russell Hay
Improve test coverage (#70)
90d6a28
* Removing code that should have been removed in a previous checkin

* another small improvement

* Adding coverage for one of the xfile edge cases

* Adding a few additional tests

* Updating based on code review feedback.

Note: assertRaises does not take a message
@chid Russell Hay
Update README.md (#71)
5fe8c9b
CLANotRequired
@t8y8
Create New Connections per #68 (#69)
8b8f351
Enable a prototypical API for creating new connections and datasources from scratch.
This uses ET to do XML manipulation, and is hard coded to "10.0" style integrated connections.

This will go away with the new editor/physical/logical model work underway, but it fixes a bug and will let us play around for now.
Russell Hay
#72 Add description to the field object (#73)
202acea
* Add description to the field object

Pull in the description if it exists on a `<column>` tag.
This change returns the entire `<desc>` tag including all subtags.
e.g.:

```
<desc>
  <formatted-text>
    <run>Total number of people in a country</run>
  </formatted-text>
</desc>
```

* improving the test code to use 'in'
Russell Hay
Cleanup samples (#74)
9126049
Russell Hay
Prepping for 0.3 release (#75)
ff372f4
* Prepping for 0.3 release

* Cleanup samples (#74)
Russell Hay
reordering changelog to move features about test improvements (#77)
7078177
@graysonarts graysonarts added this to the 2016.08 release (v0.3) milestone Aug 31, 2016
@graysonarts
graysonarts merged commit 9a098a7 into master Aug 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2016.08 release (v0.3)

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@graysonarts @t8y8 @chid
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.