Site can now be queried by name#151
Site can now be queried by name#151VathsalaAchar wants to merge 5 commits intotableau:developmenttableau/server-client-python:developmentfrom AtheonAnalytics:developAtheonAnalytics/server-client-python:developCopy head branch name to clipboard
Conversation
|
@VathsalaAchar thank you for the PR! I've tagged it as an 'enhancement' and we will take a look. I need to confirm that we want to do this via the legacy endpoint (the key=name) method, and not be leveraging native filtering, or something like that. In the mean time, please fill out the CLA, as documented here: http://tableau.github.io/contributing.html Thanks! EDIT: Please rebase this PR against 'Development' /cc @LGraber @RussTheAerialist |
|
@t8y8 thank you for the input. We sent in the CLA yesterday and I got a confirmation that it was filed for your records. I have also rebased the PR as asked. |
pushing up in-progress files
|
@VathsalaAchar it looked like you added in some work from the 'inprogress' branch. That branch is pretty old, and the permissions changes aren't relevant to your original PR (nor are they necessarily the design we want to go with) -- was that intentional? It looks like your changes are made against a Can you stash just your changes on and push them to a new branch made from |
|
This is related to #49 |
|
@t8y8 No that wasn't intentional, I'll fix the in progress and the master branch mess. Apologies for that! |
This is a Nice To Have for our end. We prefer to work with names rather than ids.