Closed (fixed)
Project:
Administer Users by Role
Version:
8.x-3.4
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
3 Aug 2024 at 21:16 UTC
Updated:
6 Dec 2024 at 11:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #4
rajan kumar@2026 commentedHii @smhar
I reviewed the code in this branch (git checkout -b '3465727-drupal-11-compatibility' --track administerusersbyrole-3465727/'3465727-drupal-11-compatibility') and checked the Drupal Upgrade Status module's persistent issue, which was attached with a screenshot. I have fixed the issue and created a Merge Request (MR).
Thank you.
Comment #5
rajan kumar@2026 commentedComment #6
riddhi.addweb commentedThe mentioned page issue is resolved, & I have also checked and it is working as expected. I am attaching the Screenshots & doing RTBC for the same.
Comment #7
pearls commented#2 - MR !8 didn't work.
Here are the logs.
Comment #8
adamps commentedGreat thanks I made a couple of comments in the MR
Comment #10
adamps commentedComment #11
pearls commentedI tested it. Patch is OK. Applied and module installed without any problems.
But I couldn't create a new user.
There's HTTP ERROR 500 , my d11 test site is currently unable to handle this request.
When I uninstalled the module ı can create a new user.
It was a quick test, no error logs. It might be due to my system environment.
Comment #13
rhezios commentedI've updated MR8, the commits contains changes for:
Comment #14
timohuismanI've tested the changes from MR11 against drupal/core 11.0.7 and it works as expected.
The steps I took;
drupal/recommended-project:^11mglaman/composer-drupal-lenientandcweagans/composer-patchesto install the patchComment #15
heddn+1 on RTBC. I added a gitlab-ci.yml file in that last commit that demonstrates that automated tests also pass green now on D11 too. LGTM.
Comment #16
adamps commentedComment #18
adamps commentedGreat thanks