- First commit to issue fork.
- @smokris opened merge request.
- 🇺🇸United States smokris Athens, Ohio, USA
I just opened https://git.drupalcode.org/project/jsonlog/-/merge_requests/7 which:
- combines the latest patches from Project Update Bot and xurizaemon
- updates the LoggerInterface::log signature to match core's RfcLoggerTrait::log
- fixes a "Deprecated function: date(): Passing null to parameter #1 ($format) of type string is deprecated" during installation
- marks the module as Drupal 10 compatible
I've briefly tested on Drupal 10 and confirmed that it's now able to log messages.
- First commit to issue fork.
- last update
over 1 year ago PHPLint Failed - last update
over 1 year ago 26 pass - 🇳🇿New Zealand RoSk0 Wellington
Hi Team,
It would be awesome to have feedback from maintainers . Merge request overall looks OK to me.
- First commit to issue fork.
- last update
about 1 year ago 26 pass - 🇳🇿New Zealand ericgsmith
I have pushed a change to keep D9/10 compatibility with the same branch. Please see the commit message for details - if the intent is to have a D10 only branch that commit can be dropped to keep the change from @smokris but for simplicity I think having a release compatible with both versions is best.
- last update
about 1 year ago 26 pass - last update
about 1 year ago 26 pass - Status changed to Needs work
about 1 year ago 4:38pm 11 October 2023 - 🇺🇸United States mradcliffe USA
but for simplicity I think having a release compatible with both versions is best.
I do not think it's possible to have a release that maintains compatibility with both Drupal 9 and Drupal 10 due to the underlying dependencies from PSR.
The revert change crashes and makes the module not compatible with Drupal 10.
- Status changed to Needs review
about 1 year ago 4:40pm 11 October 2023 - 🇺🇸United States mradcliffe USA
Oh, hmm, I seem to be using an old version of the patch. Ignore my previous comment.
- 🇸🇪Sweden emek
When can we expect a D10 compatible versoin of this module?
- 🇧🇪Belgium lorenzs
Hi, It has been a very busy year but it is my plan to get a D10 version out asap.
- last update
about 1 year ago 33 pass, 1 fail - last update
about 1 year ago 12 pass, 1 fail - Status changed to Fixed
about 1 year ago 2:54pm 13 October 2023 - 🇧🇪Belgium lorenzs
D10 compatibility is merged in the 4.x branch which will get a release. Thanks for preparing this!
Automatically closed - issue fixed for 2 weeks with no activity.