- Issue created by @Anybody
- π¬π§United Kingdom aaron.ferris
aaron.ferris β made their first commit to this issueβs fork.
- π¬π§United Kingdom aaron.ferris
aaron.ferris β changed the visibility of the branch 3437804-add-option-to to hidden.
- Merge request !6Issue #3437804: add an option to skip existing username validation β (Merged) created by aaron.ferris
- Status changed to Needs review
about 1 year ago 3:44pm 3 April 2024 - π©πͺGermany Anybody Porta Westfalica
Thanks @aaron.ferris this really looks good! I havn't had time to give it a try, but code-wise it LGTM!
- First commit to issue fork.
- π©πͺGermany Grevil
Works great! Made some minor adjustments to the code, I decided against an entry in the config install file (for now) as not a single config value is set there, and I don't want to make unrelated fixes here.
As commented in code, this module needs quite a bit of work anyway. Otherwise, this LGTM! And works as expected! Please do a final review of my changes @Anybody and then, we can set this RTBC.
PS: Already tested the update hook. Also works as expected.
- Status changed to RTBC
12 months ago 11:58am 17 April 2024 - π©πͺGermany Anybody Porta Westfalica
Great work @aaron.ferris and @Grevil! Let's get this finished. Setting priority to major, as existing users can't even change their passwords running into this!
- First commit to issue fork.
-
VladimirAus β
committed 27e0b9f4 on 8.x-1.x authored by
aaron.ferris β
Issue #3437804: Add option to exclude validation for exsting usernames
-
VladimirAus β
committed 27e0b9f4 on 8.x-1.x authored by
aaron.ferris β
- Status changed to Fixed
12 months ago 12:43pm 17 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.