From 04acda55ec86949c6e096577635549121a7f38b6 Mon Sep 17 00:00:00 2001 From: Ankush Gola <9536492+agola11@users.noreply.github.com> Date: Sun, 19 Mar 2023 20:12:33 -0700 Subject: [PATCH] Don't use dynamic api endpoint for Zapier NLA (#1803) From Robert "Right now the dynamic/ route for specifically the above endpoints is acting on all providers a user has set up, not just the provider for the supplied API key." --- langchain/utilities/zapier.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/langchain/utilities/zapier.py b/langchain/utilities/zapier.py index 0ffb04f9..64a21514 100644 --- a/langchain/utilities/zapier.py +++ b/langchain/utilities/zapier.py @@ -38,7 +38,6 @@ class ZapierNLAWrapper(BaseModel): zapier_nla_api_key: str zapier_nla_api_base: str = "https://nla.zapier.com/api/v1/" - zapier_nla_api_dynamic_base: str = "https://nla.zapier.com/api/v1/dynamic/" class Config: """Configuration for this pydantic object.""" @@ -101,7 +100,7 @@ class ZapierNLAWrapper(BaseModel): https://nla.zapier.com/api/v1/dynamic/docs) """ session = self._get_session() - response = session.get(self.zapier_nla_api_dynamic_base + "exposed/") + response = session.get(self.zapier_nla_api_base + "exposed/") response.raise_for_status() return response.json()["results"] @@ -148,8 +147,8 @@ class ZapierNLAWrapper(BaseModel): data = self.preview(*args, **kwargs) return json.dumps(data) - def list_as_str(self, *args, **kwargs) -> str: # type: ignore[no-untyped-def] + def list_as_str(self) -> str: # type: ignore[no-untyped-def] """Same as list, but returns a stringified version of the JSON for insertting back into an LLM.""" - actions = self.list(*args, **kwargs) + actions = self.list() return json.dumps(actions)