@mmd-osm commented on this pull request.
> @@ -24,9 +24,8 @@
:include_blank => t("auth.providers.none"),
:hide_label => true,
:wrapper => { :class => "col-auto mb-0" } %>
- <%= f.text_field(:auth_uid, :hide_label => true, :wrapper => { :class =>
"col mb-0" }) %>
+ <%= f.hidden_field(:auth_uid, :hide_label => true, :wrapper => { :class
=> "col mb-0" }) %>
Right, I've already removed the uid from the redirect as part of this PR.
Initially, I was a bit confused about your comment. I guess you were looking at
the master branch instead, where this change isn't in place yet.
I will remove the hidden parameter and the comparison in the update method as
well.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6038#discussion_r2106290137
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6038/review/2866928...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
