Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
CDN UI depends on config module because off Url::fromRoute('config.import_single')->toString(TRUE)->getGeneratedUrl() in CdnSettingsForm.php:144
Comment | File | Size | Author |
---|---|---|---|
#10 | interdiff-2709003-8-10.txt | 1.13 KB | Vinay15 |
#10 | 2709003-10.patch | 1.02 KB | Vinay15 |
#8 | 2709003-8.patch | 1.22 KB | Vinay15 |
add-config-dependency.patch | 253 bytes | chr.fritsch |
Comments
Comment #2
Wim LeersThanks, great catch! I think it'd be better to not depend on the
config
module though, and just let it link to the appropriate documentation on d.o instead.(Because if you have the Config module enabled, chances are you'll use configuration directly, and not the CDN UI module.)
Thoughts?
Comment #3
Wim Leers.
Comment #4
chr.fritschLinking to the docs is the better solution. Having as less as possible dependencies is always good.
Comment #5
Wim LeersIndeed!
So this would then need to link to https://www.drupal.org/documentation/administer/config. There's no more specific documentation page than that one. Care to roll a patch?
Comment #6
Vinay15Hi Wim,
Can you suggest me where should I add this link?
Comment #7
Wim Leers#6: the IS already describes where :)
Comment #8
Vinay15Thanks!! I should have read the IS properly. Anyways, I have tried to add the link.
Comment #9
Wim LeersRather than adding a new link, just update the existing link. :)
Comment #10
Vinay15Tried updating the patch.
Comment #11
Wim LeersPerfect, thanks! :)
Comment #13
Wim Leers