- π¬π§United Kingdom the_g_bomb
Patch https://git.drupalcode.org/project/logging_alerts/-/merge_requests/4.diff installs and suppresses the watchdog errors as expected.
- π¬π§United Kingdom the_g_bomb
Added D10 compatibility
https://git.drupalcode.org/project/logging_alerts/-/merge_requests/5.diffTry adding the patch with:
composer require mglaman/composer-drupal-lenient composer config --merge --json extra.drupal-lenient.allowed-list '["drupal/logging_alerts"]' composer config --json --merge extra.patches.drupal/logging_alerts '{"3190943: User deprecated function": "https://git.drupalcode.org/project/logging_alerts/-/merge_requests/5.diff"}' composer require --no-update 'drupal/logging_alerts:2.0.x-dev@dev'
or
composer config repositories.logging_alerts vcs git@git.drupal.org:issue/logging_alerts-3190943.git composer require --no-update drupal/logging_alerts:dev-3190943-add-d10-compatibility
- π³π±Netherlands eelkeblok Netherlands π³π±
Attached is a patch based on MR 4, but against 2.0.0 (the 2.0.x development branch already contains a change for this issue which breaks a patch generated from the pure diff of MR 4).
@the_g_bomb: Is this the only thing standing in the way of D10 compatibility of this module? In that case, I think it might be warranted to add this all in one go. One doubt I have is that the info.yml files claim wider core compatibility (^8...) than the composer.json (^8.8 ...). Maybe (just a suggestion) MR 4 should be merged and the D10-stuff left for a follow-up issue.
- First commit to issue fork.
-
poker10 β
committed 8d35d489 on 2.0.x authored by
sgoodwin β
Issue #3190943 by joelpittet, gruda, larisse, abu-zakham: User...
-
poker10 β
committed 8d35d489 on 2.0.x authored by
sgoodwin β
- Status changed to Fixed
about 1 year ago 11:54pm 27 March 2024 - πΈπ°Slovakia poker10
I have merged the MR 4 into 2.0.x. Let's continue with all other D10 compatibility issues here: π Drupal 10 compatibility fixes Needs review
Thanks everyone!
Automatically closed - issue fixed for 2 weeks with no activity.