Skip to content

Conversation

mcalinghee
Copy link
Contributor

@mcalinghee mcalinghee commented May 15, 2025

It is now possible to forward the login_hint with a mxid to the upstream providers thanks to #4512

This PR allows us to support the login_hint with an email when login_with_email_allowed is activated.
In particular this PR will display the login_hint ie. email on the MAS login page.
It will also forward the email login_hint to upstream providers.

@mcalinghee mcalinghee marked this pull request as draft May 15, 2025 08:47
@mcalinghee mcalinghee force-pushed the feat/login_hint_with_email branch from 86e806c to 34762a6 Compare May 15, 2025 11:32
@mcalinghee mcalinghee marked this pull request as ready for review May 15, 2025 11:36
@mcalinghee mcalinghee force-pushed the feat/login_hint_with_email branch from 34762a6 to 9c9cc3d Compare June 3, 2025 13:15
@mcalinghee mcalinghee force-pushed the feat/login_hint_with_email branch 3 times, most recently from 0a1c9a2 to 52bb16b Compare June 5, 2025 13:41
@mcalinghee mcalinghee requested a review from sandhose June 5, 2025 13:42
@mcalinghee mcalinghee marked this pull request as draft June 5, 2025 16:09
@mcalinghee mcalinghee force-pushed the feat/login_hint_with_email branch from 52bb16b to 07ac7ca Compare June 5, 2025 16:17
@mcalinghee mcalinghee marked this pull request as ready for review June 5, 2025 16:18
@mcalinghee mcalinghee force-pushed the feat/login_hint_with_email branch 7 times, most recently from 2e23003 to 28a15d4 Compare June 12, 2025 14:14
@mcalinghee mcalinghee force-pushed the feat/login_hint_with_email branch from 28a15d4 to d18fcaa Compare June 16, 2025 10:11
@mcalinghee mcalinghee requested a review from MatMaul June 16, 2025 13:39
@odelcroi odelcroi force-pushed the feat/login_hint_with_email branch from d18fcaa to a5d3a2f Compare July 30, 2025 06:51
@mcalinghee mcalinghee changed the title email login_hint support when login_with_email_allowed is activated forward email login_hint to upstream providers Jul 30, 2025
@mcalinghee mcalinghee changed the title forward email login_hint to upstream providers support email login_hint Jul 30, 2025
@mcalinghee mcalinghee force-pushed the feat/login_hint_with_email branch from 9b1c45f to cdf53ca Compare July 30, 2025 13:19
@sandhose
Copy link
Member

@mcalinghee after an initial review, I would appreciate if you don't force-push to the branch, as it often makes GitHub confused and makes it harder to understand what changed. If you ever need to update from the main branch, prefer merging it into your branch instead of rebasing :)

@mcalinghee mcalinghee requested a review from sandhose July 31, 2025 11:12
Copy link
Member

@sandhose sandhose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! I've pushed a few cosmetic fixes myself, but this looks good to me now!

@sandhose sandhose enabled auto-merge August 18, 2025 14:52
@sandhose sandhose changed the title support email login_hint Support handling email login_hint Aug 18, 2025
@sandhose sandhose changed the title Support handling email login_hint Support suggesting email as login_hint Aug 18, 2025
@sandhose sandhose merged commit b3d8e11 into element-hq:main Aug 18, 2025
19 checks passed
@sandhose sandhose added A-Login-Flow Related to the user login flow T-Enhancement New feature of request labels Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Login-Flow Related to the user login flow T-Enhancement New feature of request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants