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.

Sign up for GitHub

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

Jump to bottom

langchain[patch]: Take all required fields into account for OpenAPI chain #6700

Merged
merged 2 commits into from
Sep 5, 2024
Merged

langchain[patch]: Take all required fields into account for OpenAPI chain #6700

merged 2 commits into from
Sep 5, 2024

Conversation

ikalachy
Copy link
Contributor

@ikalachy ikalachy commented Sep 5, 2024 •
edited
Loading

Fixes #6699

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Sep 5, 2024
@vercel Vercel
Copy link

vercel bot commented Sep 5, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Sep 5, 2024 5:05pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Sep 5, 2024 5:05pm

@dosubot dosubot bot added the auto:bug Related to a bug, vulnerability, unexpected error with an existing feature label Sep 5, 2024
@bracesproul bracesproul changed the title fix: take into account all required fields for Bug #6699 langchain[patch]: Take all required fields into account for OpenAPI chain Sep 5, 2024
@bracesproul bracesproul merged commit f0a6094 into langchain-ai:main Sep 5, 2024
32 checks passed
This was referenced Sep 12, 2024
Open
Open
@bruno353 bruno353 mentioned this pull request Sep 13, 2024
Open
This was referenced Sep 14, 2024
Open
Open
Open
@bruno353 bruno353 mentioned this pull request Sep 17, 2024
Open
This was referenced Oct 3, 2024
Open
Open
@lholmquist lholmquist mentioned this pull request Oct 3, 2024
Closed
@davidebukali davidebukali mentioned this pull request Oct 3, 2024
Open
This was referenced Oct 5, 2024
Closed
Merged
@langchain-ai langchain-ai locked as spam and limited conversation to collaborators Oct 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
auto:bug Related to a bug, vulnerability, unexpected error with an existing feature size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

convertOpenAPISchemaToJSONSchema doesnt take into account required params
2 participants
@ikalachy @bracesproul