[openstreetmap/openstreetmap-website] Re-arrange login and signup screens as discussed in #4128 (PR #4455)

Milan Cvetkovic notifications at github.com
Thu Jan 11 11:09:42 UTC 2024


@milan-cvetkovic commented on this pull request.



>          redirect_to auth_url(current_user.auth_provider, current_user.auth_uid), :status => :temporary_redirect
       else
         # Save the user record
-        session[:new_user] = current_user.attributes.slice("email", "display_name", "pass_crypt")
-        redirect_to :action => :terms
+        session[:new_user] = current_user.attributes.slice("email", "display_name", "pass_crypt", "consider_pd")

Re-arranged the code not to use `session[:new_user]`

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4455#discussion_r1448683556
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4455/review/1815282304 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240111/198c9f88/attachment.htm>


More information about the rails-dev mailing list