8000 #141 Enable Zip64 Extensions To Support Zipfile >= 2GB by doulam · Pull Request #143 · tableau/document-api-python · GitHub
[go: up one dir, main page]

Skip to content

#141 Enable Zip64 Extensions To Support Zipfile >= 2GB #143

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

Closed
wants to merge 8 commits into from

Conversation

doulam
Copy link
Contributor
@doulam doulam commented Aug 3, 2017

I enabled the option AllowZip64=True for ZipFile() in the xfiles.py. This allows documentapi to handle tdsx/twbx that is greater than 2GB.

Russell Hay and others added 8 commits November 1, 2016 14:50
* Fix tableau#117 by only attempting files with the right extension inside the archive (tableau#118)

* Commenting and Docstring cleanup. A few very small code cleanups (tableau#120)

Add docstrings and remove clutter. I also made some very tiny tweaks to some code for clarity.

* Small cleanups for various editors. Play nice with built in test-runners (tableau#121)

* Add Py36, update travis to use pycodestyle (tableau#124)

* Add `initial sql` and `query band` support (tableau#123)

Addresses tableau#109 and tableau#110

* Prep for release of 0.6 (tableau#125)

* Prep for release of 0.6

* wordsmithing the changelog
Copy link
Contributor
t8y8 commented Aug 4, 2017

Hello! Thanks for the contribution.

A few things we need to take care of:

  • Can you sign and submit the CLA?
  • This looks like it was forked from master, can you rebase on top of development and resubmit?

I did a quick google, and this was enabled by default in python 3.4, but since we support 2.7, I think it's a simple enough change to make.

@doulam
Copy link
Contributor Author
doulam commented Aug 7, 2017

@t8y8 Sure, let me rebase this on the development branch. Do you have quick instructions on how to sign github CLA? Never done this before. Thanks.

@doulam
Copy link
Contributor Author
doulam commented Aug 7, 2017

@t8y8 nvm. I found the Tableau CLA on the Tableau Github. Will attach it on my update. Thanks.

@doulam doulam closed this Aug 7, 2017
@doulam
Copy link
Contributor Author
doulam commented Aug 7, 2017

Close this PR. Going to resubmit a new one from the development branch.

6D80

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.

3 participants
0