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.

Sign up for GitHub

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

Jump to bottom

Moving and Copying folder #1619

Open
tinect opened this issue Jan 2, 2023 · 2 comments
Open

Moving and Copying folder #1619

tinect opened this issue Jan 2, 2023 · 2 comments

Comments

@tinect
Copy link
Contributor

tinect commented Jan 2, 2023

Feature Request

Flysystem v3 should support moving and copying folders by default.
Currently, also not possible with S3.
See also: #1482
@keulinho
Copy link
Contributor

keulinho commented Jan 5, 2023

An alternative solution might be adding explicit moveDirectory() and copyDirectory() methods, similar to the existing delete() and deleteDirectory() methods as Object storage like s3 don't really have "directories", so you always need to enumerate the objects in the given folder (which is only a filename prefix).

By splitting the methods for simple files/objects and directories/prefixes you allow the adapters to implement it in a way that is optimal for that specific adapter.

@tinect
Copy link
Contributor Author

tinect commented Jan 5, 2023

By splitting the methods for simple files/objects and directories/prefixes you allow the adapters to implement it in a way that is optimal for that specific adapter.

Uhhh, I like this idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tinect @keulinho