- ๐ฎ๐ณIndia yogeshsevak
Providing updated patch with fixes. Please review.
- ๐ฆ๐บAustralia gumnut
Tested #3 and it works, this patch is ready to merge
- Status changed to Needs work
over 1 year ago 12:07pm 22 August 2023 - ๐ฌ๐งUnited Kingdom pobster
The patch at #3 ๐ Automated Drupal 10 compatibility fixes RTBC needs work, it should be using DI in class files - and not reference the services directly.
- ๐ฎ๐นItaly finex
The module also throws many other errors, like when calling "Unicode::strtolower" or entering into the settings. I will open some new issues on the next days.
- ๐บ๐ธUnited States Albert Volkman
Albert Volkman โ made their first commit to this issueโs fork.
- Status changed to Needs review
about 1 year ago 6:10pm 19 October 2023 - Status changed to RTBC
about 1 year ago 8:49am 6 November 2023 - First commit to issue fork.
- ๐บ๐ธUnited States wmfinck
Subscribing... I hope there is a D10 version.
- ๐ฌ๐งUnited Kingdom unqunq
Hi, I'd like this merged please. My employer needs this done in order to approve the use of this module in Drupal 10. Thank you.
- Assigned to UsingSession
-
UsingSession โ
committed 9797c917 on 2.x
Issue #3296984 by Albert Volkman, mlncn, yogeshsevak, UsingSession:...
-
UsingSession โ
committed 9797c917 on 2.x
- ๐ฌ๐งUnited Kingdom pobster
You've merged the wrong one ... You've picked the patch which doesn't use any Dependency Injection...
- Status changed to Needs work
6 months ago 10:37am 28 June 2024 - ๐บ๐ฆUkraine UsingSession
@pobster Thanks for the reply, but this PR is no longer relevant, as changes have already been made regarding Dependency Injection
- Status changed to Fixed
6 months ago 3:54pm 28 June 2024 Automatically closed - issue fixed for 2 weeks with no activity.