@rkoeze commented on this pull request.
>
- def confirm
- if request.post?
- user = User.find_by_token_for(:new_user, params[:confirm_string])
+ def show
+ if params[:confirm_string]
+ user = User.lookup_by_confirmation_token(params[:confirm_string])
if !user
This was one of the bigger changes in terms of UX. Now if a user can't be found
or their account has already been confirmed, they are redirected to the root
path in both the new account and email change cases.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5895#pullrequestreview-2745427245
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5895/review/2745427...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
