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

add async to zapier nla tools #6791

Merged
merged 5 commits into from
Jun 27, 2023

Conversation

mplachter
Copy link
Contributor

@mplachter mplachter commented Jun 27, 2023

Replace this comment with:

  • Description: Add Async functionality to Zapier NLA Tools
  • Issue: n/a
  • Dependencies: n/a
  • Tag maintainer:

Maintainer responsibilities:

If no one reviews your PR within a few days, feel free to @-mention the same people again.

See contribution guidelines for more information on how to write/run tests, lint, etc: https://github.com/hwchase17/langchain/blob/master/.github/CONTRIBUTING.md

@vercel
Copy link

vercel bot commented Jun 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Jun 27, 2023 7:01pm

@mplachter mplachter marked this pull request as draft June 27, 2023 00:09
@mplachter mplachter marked this pull request as ready for review June 27, 2023 00:17
@mplachter
Copy link
Contributor Author

It looks like it might overlap with #3392 but looks like this one also implements the list and the preview async stuff.

@dev2049 dev2049 added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Jun 27, 2023
@mplachter
Copy link
Contributor Author

@agola11 @vowelparrot

Had a merge conflict pop up.

Looking for a review when you get a chance please and thank you. :)

@hwchase17 hwchase17 merged commit d6664af into langchain-ai:master Jun 27, 2023
vowelparrot pushed a commit that referenced this pull request Jul 4, 2023
Replace this comment with:
  - Description: Add Async functionality to Zapier NLA Tools
  - Issue:  n/a 
  - Dependencies: n/a
  - Tag maintainer: 

Maintainer responsibilities:
  - Agents / Tools / Toolkits: @vowelparrot
  - Async: @agola11

If no one reviews your PR within a few days, feel free to @-mention the
same people again.

See contribution guidelines for more information on how to write/run
tests, lint, etc:
https://github.com/hwchase17/langchain/blob/master/.github/CONTRIBUTING.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm PR looks good. Use to confirm that a PR is ready for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants