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

Feature/unified batches api #792

Closed
wants to merge 31 commits into from
Closed

Conversation

narengogi
Copy link
Collaborator

@narengogi narengogi commented Dec 5, 2024

Testing done:

Related Issues: (optional)

some refactoring

file upload handler for /v1/files route

rename gatewayRequestBody to gatewayRequestBodyJSON and pass raw body as a parameter to getEndpoint as gatewayRequestBody

files upload handler for cohere

changes to handle get routes

list files for cohere

handle get file route

Get file for cohere

delete file handler

delete file for cohere

get file content handler

refactoring

revert requestValidator changes

refactoring

some refactoring

some refactoring

formatting

minor fixes

pass url instead of path because query parameters are required at some places
@narengogi narengogi force-pushed the feature/unified-batches-api branch from acdcee9 to bdc6839 Compare December 6, 2024 12:58
@narengogi narengogi force-pushed the feature/unified-batches-api branch from bd35faf to 0ad05b4 Compare December 8, 2024 08:07
@narengogi narengogi force-pushed the feature/unified-batches-api branch from c5d6a3f to d79b22b Compare December 13, 2024 06:49
@narengogi narengogi requested review from VisargD and b4s36t4 December 17, 2024 12:51
@narengogi narengogi force-pushed the feature/unified-batches-api branch from 0fa48ff to 74095ff Compare January 2, 2025 07:33
@narengogi
Copy link
Collaborator Author

closing this PR to make changes on a fresh branch

@narengogi narengogi closed this Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant