Skip to content
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

simple wrapper for tooling api requests #54

Closed
wants to merge 3 commits into from

Conversation

vnehess
Copy link

@vnehess vnehess commented Jan 27, 2015

simple copy of the restful() method with the tooling API path added for connivence.

@hello-josh
Copy link
Contributor

This is pretty useful. Could you add **kwargs to tooling() and pass it through to self.request.get() the same way the _call_salesforce() methods do? The restful() method, as well as a few others, are missing it.

@vnehess
Copy link
Author

vnehess commented Jan 28, 2015

Ok
Sure

On Jan 27, 2015, at 11:03 AM, Josh Johnston notifications@github.com wrote:

This is pretty useful. Could you add **kwargs to tooling() and pass it through to self.request.get() the same way the _call_salesforce() methods do? The restful() method, as well as a few others, are missing it.


Reply to this email directly or view it on GitHub.

@@ -202,6 +205,27 @@ def restful(self,path,params):
return None
else:
return json_result

# Generic Tooling API Function
def tooling(self,path,params):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add spaces between the arguments here?

@demianbrecht
Copy link
Collaborator

This is super old, but would still be nice to get in if tests were added.

@nullglitch
Copy link

can anyone with write access merge this branch with the master? The tooling function is a must have which is missing in the master.

@demianbrecht
Copy link
Collaborator

@nullglitch why is this function a must have? It's a convenience version of restful.

@tanderegg
Copy link
Collaborator

@nullglitch It looks like there are a few outstanding issues with this PR, if someone wants to address them then I can merge the PR. See @nickcatal's comments under "Files Changed", plus @bolaurent's error report.

@jon-wobken jon-wobken added the Backlog Issue will be looked at sometime in the future label Dec 7, 2020
@amd-yan
Copy link
Contributor

amd-yan commented Jan 27, 2021

@nullglitch why is this function a must have? It's a convenience version of restful.

I had the same same question until I realized that there are certain things can only be updated using tooling API today. Eg: Global Picklist.

@amd-yan amd-yan mentioned this pull request Jan 27, 2021
@amd-yan
Copy link
Contributor

amd-yan commented Jan 27, 2021

I spent sometime this afternoon and added tooling API to the latest code. It works to get my job done. I also created a pull request, and hopefully, other people may find it useful too.

@jon-wobken jon-wobken closed this Mar 31, 2021
@jon-wobken
Copy link
Collaborator

Resolved by #445

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backlog Issue will be looked at sometime in the future enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants