-
Notifications
You must be signed in to change notification settings - Fork 445
add export_wb sample for 'fullpdf' #264
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
add export_wb sample for 'fullpdf' #264
Conversation
samples/export_wb.py
Outdated
| parser.add_argument('--server', '-s', required=True, help='server address') | ||
| parser.add_argument('--username', '-u', required=True, help='username to sign into server') | ||
| parser.add_argument('--site', '-S', default=None) | ||
| parser.add_argument('-p', default=None) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add --password
samples/export_wb.py
Outdated
| parser.add_argument('--logging-level', '-l', choices=['debug', 'info', 'error'], default='error', | ||
| help='desired logging level (set to error by default)') | ||
| parser.add_argument('--file', '-f', help='filename to store the exported data') | ||
| parser.add_argument('resource_id', help='LUID for the view') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"for the workbook"
samples/export_wb.py
Outdated
|
|
||
| parser.add_argument('--logging-level', '-l', choices=['debug', 'info', 'error'], default='error', | ||
| help='desired logging level (set to error by default)') | ||
| parser.add_argument('--file', '-f', help='filename to store the exported data') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add to the help that this is the default or just make it explicitly the default
samples/export_wb.py
Outdated
| server = TSC.Server(args.server, use_server_version=True) | ||
| try: | ||
| with server.auth.sign_in(tableau_auth): | ||
| views = list(TSC.Pager(server.views)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems dumb (Russell told me to put that :) )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still need to address the dumb.
…hrowing it away and getting it again...
t8y8
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Emulating the basic functionality of fullpdf from tabcmd.