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

feat(blob): add createFolder method #751

Merged

Conversation

vvo
Copy link
Member

@vvo vvo commented Sep 13, 2024

And remove variadic calls from main put methods, which will allow us to more
easily upgrade to newer TS versions.

Related:

And remove variadic calls from main put methods, which will allow us to more
easily upgrade to newer TS versions.

Related:
- #682
- #667
@vvo vvo requested review from luismeyer and a team September 13, 2024 14:45
@vvo vvo self-assigned this Sep 13, 2024
Copy link

vercel bot commented Sep 13, 2024

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

Name Status Preview Updated (UTC)
vercel-storage-next-integration-test-suite ✅ Ready (Inspect) Visit Preview Sep 13, 2024 3:13pm

Copy link

changeset-bot bot commented Sep 13, 2024

🦋 Changeset detected

Latest commit: 7174601

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@vercel/blob Minor
vercel-storage-integration-test-suite Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

}

/**
* Creates a folder in your store. Vercel Blob has no real concept of folders, our file browser on Vercel.com displays folders based on the presence of trailing slashes in the pathname. Unless you are building a file browser system, you probably don't need to use this method.
Copy link
Member

@luismeyer luismeyer Sep 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we warn here, that folders will appear as blobs when you call list()? #640

listFolders(),
multipartNodeJsFileStream(),
fetchExampleMultipart(),
// textFileExample(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is for testing I guess

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes will remove

@@ -54,14 +43,14 @@ export function createPutMethod<TOptions extends PutCommandOptions>({
const options = await createPutOptions({
pathname,
// when no body is required (for folder creations) options are the second argument
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this can be removed

@vvo vvo merged commit 8098803 into main Sep 13, 2024
11 of 12 checks passed
@vvo vvo deleted the vincentvoyer/sto-1482-add-createfolder-and-remove-variadic-calls branch September 13, 2024 15:11
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