Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

Feature aliyun-oss Filesystem #939

Merged
merged 6 commits into from
Jun 21, 2019
Merged

Feature aliyun-oss Filesystem #939

merged 6 commits into from
Jun 21, 2019

Conversation

benydc
Copy link
Contributor

@benydc benydc commented May 8, 2019

Requirements:
composer require aliyuncs/aliyun-oss-flysystem

@benydc benydc mentioned this pull request May 8, 2019
@benhaynes
Copy link
Member

Thank you @benydc !

My only concern is that it seems the way this is structured now, all storage options need to be in core, including any dependencies. @BJGajjar, is there a way we could have storage adapters be extensions (like they're supposed to be)?

We have requests for many other storage options, but I worry about adding them all to core of that increases the file size or makes the config file too complex. I'd like to have an easy way to add external adapters by copying in a self contained directory, just like or other extensions.

Any ideas?

@binal-7span binal-7span merged commit 758b6b5 into directus:master Jun 21, 2019
samvasko pushed a commit to samvasko/api that referenced this pull request Nov 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants