Updating the info file to include the core_version_requirement.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | d9_patch_ckeditorheight.png | 24.54 KB | saschaeggi |
| #2 | 3105304-2-drupal9-compatibility.patch | 418 bytes | saschaeggi |
Updating the info file to include the core_version_requirement.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | d9_patch_ckeditorheight.png | 24.54 KB | saschaeggi |
| #2 | 3105304-2-drupal9-compatibility.patch | 418 bytes | saschaeggi |
Comments
Comment #2
saschaeggiAdding the patch
Comment #3
saschaeggiTested, passed.

Set this RTBC :)
Comment #4
saschaeggiComment #5
geek-merlin> Tested, passed.
Which tool did you use? drupal-check?
Does it make sense to include the drupalci.yml mentioned here? https://github.com/mglaman/drupal-check/wiki/Drupal-9-Readiness
Comment #6
saschaeggi@geek-merlin
I've checked it with upgrade-status: https://www.drupal.org/project/upgrade_status
I don't think it's needed, but I'd rather open a new ticket to explore it if it's necessary for this module.
Cheers
Comment #7
geek-merlinAh thanks, i was not aware of this yet. Not that i expected something else as the module is trivial PHP-wise and all the flesh is in JS.
So putting this on the pipeline to commit. Thanks a bunch!
Comment #9
geek-merlin