-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue with capitalization when humanizing authentication keys in failure flash message #4900
Comments
I'm pretty sure this is the line that would need to be changed. https://github.com/plataformatec/devise/blob/master/lib/devise/failure_app.rb#L106 |
I agree with the OP fwiw - this isn't the behaviour I'd expect, all the more so for the fact that it changed in an earlier version |
This issue is still present in 4.6.2. Was it intentionally merged into the 5-rc branch? |
@archonic Yes - because this change does not preserve backwards compatibility. We are going to release in version 5 though. |
Thank you so much for building something that I use in almost every rails project I start.
Environment
Current behavior
When looking at the
devise.failure.not_found_in_database
anddevise.failure.invalid
error messages, it currently says "Invalid Email or password." when theauthentication_keys
=:email
. To me this is strange capitalization. I'm not expecting "Email" to be capitalized.Expected behavior
I like that this takes the
authentication_keys
and makes them human readable, but I wish that it would make them lower case so that it read, "Invalid email or password." instead.The text was updated successfully, but these errors were encountered: