[openstreetmap/openstreetmap-website] Remove legacy OpenID login (PR #6038)

[mmd]

@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/2866928971 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250525/3bfad039/attachment.htm>