Currently we're doing this at the form level. What do you think about doing this at the authentication service level so that anything that authenticates with the "user.auth" service is allowed to use email as well.

This would also solve the issue with the module not working with the REST api endpoint as noted in #2981315: Only overrides login form, REST login does not accept emails.

CommentFileSizeAuthor
#4 interdiff.txt868 bytesgrndlvl
#4 3122968-3.patch2.64 KBgrndlvl
#2 3122968-2.patch2.78 KBgrndlvl

Comments

grndlvl created an issue. See original summary.

grndlvl’s picture

StatusFileSize
new2.78 KB
grndlvl’s picture

Status: Active » Needs review
grndlvl’s picture

StatusFileSize
new2.64 KB
new868 bytes

Minor clean up to remove password service completely since we're not using it.

mqanneh’s picture

@grndlvl
Moving this into new branch 8.x-2.x

mqanneh’s picture

Version: 8.x-1.x-dev » 8.x-2.x-dev
mqanneh’s picture

Status: Needs review » Fixed
mqanneh’s picture

Status: Fixed » Closed (fixed)