@tomhughes commented on this pull request.
> @@ -14,6 +14,7 @@ class ConfirmationsController < ApplicationController
before_action :require_cookies, :only => [:confirm]
def confirm
+ # The post happens in confirm.html.erb and user.js
I don't think this is very useful - even if you accept it's true right now, and
that depends on how you define "happens" exactly, there's no guarantee that
those locations will stay the same and nobody will find and update this if they
change one of those files.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6138#pullrequestreview-2967198355
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6138/review/2967198...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
