Skip to content

Conversation

@xsrf
Copy link
Contributor

@xsrf xsrf commented Oct 18, 2024

Enable autocomplete for Username Form

This allows browsers and password managers to suggest a username.

Closes #34091

@xsrf xsrf requested review from a team as code owners October 18, 2024 13:43
@codespearhead
Copy link

codespearhead commented Oct 18, 2024

It looks like you forgot to sign off the commit [1], as shown here [2].

To fix this, run:

git clone --single-branch --branch feature_autocomplete [email protected]:xsrf/keycloak.git
git commit --amend --signoff -m "Enable autocomplete for Username Form" -m "This allows browsers and password managers to suggest a username." -m "Closes https://github.com/keycloak/keycloak/issues/34091"
git push -f

Also, I recommend updating your PR description to match the commit message, for the reasons mentioned here [3].


[1] https://github.com/keycloak/keycloak/blob/main/CONTRIBUTING.md#developers-certificate-of-origin
[2] #30669
[3] #30669 (comment)

This allows browsers and password managers to suggest a username.

Closes keycloak#34091

Signed-off-by: xsrf <[email protected]>
@xsrf xsrf force-pushed the feature_autocomplete branch from 95b82f7 to 668940f Compare October 18, 2024 15:31
@jonkoops jonkoops enabled auto-merge (squash) October 18, 2024 16:22
@jonkoops jonkoops merged commit 47a7d9c into keycloak:main Oct 18, 2024
jonkoops pushed a commit to jonkoops/keycloak that referenced this pull request Dec 3, 2024
jonkoops added a commit that referenced this pull request Dec 3, 2024
Closes #34091

Signed-off-by: xsrf <[email protected]>
(cherry picked from commit 47a7d9c)

Co-authored-by: Andreas <[email protected]>
@edewit edewit mentioned this pull request Dec 11, 2024
edewit pushed a commit to edewit/keycloak that referenced this pull request Dec 11, 2024
…35543)

Closes keycloak#34091

Signed-off-by: xsrf <[email protected]>
(cherry picked from commit 47a7d9c)

Co-authored-by: Andreas <[email protected]>
Signed-off-by: Erik Jan de Wit <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Username Form should support autocomplete

3 participants