Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@mamhoff
Copy link
Contributor

@mamhoff mamhoff commented Oct 10, 2025

There are a few important additions here, notably the change to use the new LocaleSelect.

tvdeyen added 11 commits October 2, 2025 23:58
Need to reload routes in controller tests, because of
an unresolved issue with Devise
Devise has a email presence validation, but allows
to override it. We make it conditional by using the
devise aithentication keys.
Turbo requires a 422 response in order to show validation
errors.
We are using the resource_table partial
- Only shows locale select if many availabel locales exist
- Set login and email fields to be required accordingly
- Add autocomplete attribute where appropriate
- Require email / login field
- Add proper autocomplete values
Require fields and add appropriate autocomplete values.
Login, Signup and Password reset forms improvements
@tvdeyen tvdeyen merged commit 80c0dc3 into 8.0-stable Oct 10, 2025
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.