[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