We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Right now we are configuring the system like this:
filesystems: s3_lib: adapter: s3_lib visibility: public cache: local
Then in the DI Extension we have to check manually:
$options = []; if (array_key_exists('visibility', $config)) { $options['visibility'] = $config['visibility']; }
This will get troublesome later when new options are available etc. I suggest we change it to this:
filesystems: s3_lib: adapter: s3_lib options: visibility: public cache: local
We can then easily use $config['options'] array in our DI Extension.
The text was updated successfully, but these errors were encountered:
Indeed. visibility seems to be given to the filesystem as an element in an option array, so this definitely makes sense to me.
visibility
Sorry, something went wrong.
No branches or pull requests
Right now we are configuring the system like this:
Then in the DI Extension we have to check manually:
This will get troublesome later when new options are available etc. I suggest we change it to this:
We can then easily use $config['options'] array in our DI Extension.
The text was updated successfully, but these errors were encountered: