[openstreetmap/openstreetmap-website] Re-arrange login and signup screens as discussed in #4128 (PR #4455)
Anton Khorev
notifications at github.com
Sun Apr 21 16:37:59 UTC 2024
@AntonKhorev commented on this pull request.
> @@ -290,7 +243,8 @@ def auth_success
failed_login t("sessions.new.auth failure")
end
else
- redirect_to :action => "new", :nickname => name, :email => email,
+ verified_email = UsersController.message_hmac(email) if email_verified && email
`verified_email` is no longer a email address and it's named `email_hmac` elsewhere. Should it be renamed here too?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4455#pullrequestreview-2013477848
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4455/review/2013477848 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240421/2f326fa1/attachment.htm>
More information about the rails-dev
mailing list