Skip to content

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

@Nutlope Nutlope force-pushed the yogish/upload-files branch from 31173e3 to 2b2b843 Compare November 5, 2024 15:38
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 force-pushed the yogish/upload-files branch from 2b2b843 to f726aba Compare November 5, 2024 15:41
@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.

3 participants