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

extra_body #256

Merged
merged 3 commits into from
Dec 11, 2024
Merged

extra_body #256

merged 3 commits into from
Dec 11, 2024

Conversation

csgulati09
Copy link
Collaborator

@csgulati09 csgulati09 commented Dec 7, 2024

Title: Handle the case of extra_body

Description:

Detailed change

  • Adding additional case of checking if kwargs are present or not

Motivation:*
Issue persists, but now in sync with OpenAI Issue

Related Issues:
Fixes: #255

@csgulati09 csgulati09 self-assigned this Dec 7, 2024
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. auto:bug labels Dec 7, 2024
@csgulati09 csgulati09 requested a review from VisargD December 11, 2024 14:25
@dosubot dosubot bot added the lgtm label Dec 11, 2024
@csgulati09 csgulati09 merged commit 5c723ec into main Dec 11, 2024
6 checks passed
@csgulati09 csgulati09 deleted the fix/retrieveCalls branch December 11, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto:bug lgtm size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Handling extra_body in retrieve calls
2 participants