- 🇫🇮Finland tuutti
We've been using this patch for a couple of months with D10 now.
- First commit to issue fork.
- Status changed to Needs work
about 1 year ago 6:33am 19 October 2023 - 🇫🇮Finland tuutti
Nevermind.
This module has a dependency to
league/flysystem-azure-blob-storage 1.0.0
and it requiresguzzlehttp/psr7 ^1.5
. while core requiresguzzlehttp/psr7 ^2.4.5
. - 🇷🇴Romania radu.cocian
I have the same issue tuutti mentioned. Is it possible to make it work with D10?
Thanks!
- 🇫🇮Finland tuutti
We ended up forking the
AzureBlobStorageAdapter
class from upstream and removing theleague/flysystem-azure-blob-storage
dependency altogether since the 1.x version is not very likely to receive any updates.The latest version requires flysystem 3.x, but drupal/flysystem only supports 1.x at the moment.
- 🇷🇴Romania radu.cocian
Thanks tuutti, that's very helpful. I tried to use your branch and install it locally on my dev server but I do something wrong and not sure what because the module's requirements are not installed (when set to load it from local path). When I define it as a git branch since there is no type in module's composer.json, is installed as a library and not under web/modules/contrib.
- 🇷🇴Romania radu.cocian
Nevermind. I was able to install it with composer / custom git repo by adding a new entry under installer-paths specific for this package
"installer-paths": {
....
"web/modules/contrib/{$name}/": [
"drupal/flysystem_azure"
],
...
}I will update the source once the pull request is approved.
Thank you!
- 🇫🇮Finland tuutti
Ah, the type is probably populated by packages.drupal.org (https://packages.drupal.org/files/packages/8/p2/drupal/flysystem_azure~d...) if it does not exist and that's why the original module works but the fork does not.
It should work now.
-
BryanLund →
committed 2d6b397a on 1.0.x authored by
tuutti →
Resolve #3287639 "Automated drupal 10"
-
BryanLund →
committed 2d6b397a on 1.0.x authored by
tuutti →
- 🇬🇧United Kingdom BryanLund
Sorry for the super late merge, but i've merged that in now @tuutti thanks!