[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