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

Adding upload API as a helper function #68

Merged
merged 4 commits into from
Nov 5, 2024
Merged

Conversation

yogishbaliga
Copy link
Contributor

@yogishbaliga yogishbaliga commented Oct 21, 2024

Need to figure out how to make it part of Together.files package

Adding upload file support for together.files interface.

@Nutlope Nutlope changed the title adding upload API as a helper function. Need to figure out how to mak… Adding upload API as a helper function Oct 22, 2024
@yogishbaliga
Copy link
Contributor Author

can this be merged now?

@Nutlope
Copy link
Collaborator

Nutlope commented Oct 23, 2024

can this be merged now?

Working on it right now, making changes based on Stainless's feedback, then will test

yogishbaliga and others added 3 commits November 5, 2024 10:40
…e it part of Together.files package

Adding upload file support for together.files interface.

removed the tests that require S3 credentials

fixing lint errors

added example & simplified upload api

removed tsignore

integrated it with our api

deleted unused file

working version

working version

validate the content of the file being uploaded

move upload out of files API and make it a standalone one

cleaned up API
@Nutlope Nutlope self-requested a review November 5, 2024 15:43
@Nutlope Nutlope merged commit ed8622e into main Nov 5, 2024
4 checks passed
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.

2 participants