- Issue created by @project update bot
This is an automated patch generated using Upgrade Status and Drupal Rector. Please see the issue summary for more details. A merge request is also openend and updated.
It is important that any automated tests available are run and that you manually test the changes.
Drupal 11 Compatibility
According to the Upgrade Status module → these changes make this module compatible with Drupal 11! 🎉
Therefore these changes update theinfo.yml
file for Drupal 11 compatibility.Leaving this issue open, even after committing the current patch, will allow the Project Update Bot → to post additional Drupal 11 compatibility fixes as they become available in Drupal Rector.
Debug info
Bot run #11-121090This patch was created using these packages:
- drupal/upgrade_status: 4.1.0
- mglaman/phpstan-drupal: 1.2.7
- palantirnet/drupal-rector: 0.20.1
- Status changed to RTBC
about 2 months ago 2:09am 10 October 2024 - 🇦🇺Australia pameeela
Manually tested this on 11.x and the module works as expected, so I think this can be merged!
- First commit to issue fork.
- Merge request !6Issue #3431737 by vladimiraus, akulsaxena, pameeela, mably: Drupal 11 compatibility fixes for login_emailusername → (Merged) created by VladimirAus
- 🇦🇺Australia VladimirAus Brisbane, Australia
- Created 2.1.x dev branch
- Added CI
- Removed version 8 support
- Removed D12 deprecations (which also resulted in updating functions from
UserAuthenticationController
as flood control was broken).
@pameeela if you can give it another test, I'd appreciate it.
Seems like some functionality was broken by code not updated from UserAuthenticationController. - 🇦🇺Australia VladimirAus Brisbane, Australia
See change record → for deprecation.
- 🇦🇺Australia pameeela
Manually tested with the latest changes:
- ✅ Logging in via email and username
- ✅ Resetting password via email and username
- ✅ Login blocked after 5 failed attempts when logging in via email
This should be all set except there are cspell and phpcs fails.
- 🇮🇳India akulsaxena
Solved all PHPCS and cspell errors
Added compatibility with Drupal ^12 to solve upgrade status pipeline failing when using Drupal 11
All pipelines passed with no errors or warnings - 🇮🇳India akulsaxena
Hi @vladimiraus, @rjjakes and @devdesagar
Since all the errors have been fixed and the issue has been completed and moved to RTBC, can we move this issue to fixed now? - 🇦🇺Australia pameeela
Just noting we currently have this module included in Drupal CMS. We're going to have to remove all modules that are not D11 compatible by our beta release on Nov 20. This one is easy to add back since there is no config, but figured it was worth mentioning.
- First commit to issue fork.
- 🇮🇳India akulsaxena
Hi @mably
I checked the MR and it looks good to me. All the pipelines are passing without any warnings. -
mably →
committed 2ee82dc3 on 3.0.x authored by
vladimiraus →
Issue #3431737 by vladimiraus, akulsaxena, pameeela, mably: Drupal 11...
-
mably →
committed 2ee82dc3 on 3.0.x authored by
vladimiraus →
- 🇫🇷France mably
3.0.0-rc1 → version, compatible with Drupal 11, has been release!
Thanks to all!
Automatically closed - issue fixed for 2 weeks with no activity.