[openstreetmap/openstreetmap-website] Already logged in warning (PR #5957)

Anton Khorev notifications at github.com
Sat Apr 26 18:27:15 UTC 2025


Fixes #5441 by showing a warning as described in https://github.com/openstreetmap/openstreetmap-website/issues/5441#issuecomment-2563121432.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5957

-- Commit Summary --

  * Move OpenID button test to user login tests
  * Warn on login page if already logged in
  * Show referer link in already logged in warning
  * Preserve anchor in referer link inside warning
  * Show only safe links inside warning

-- File Changes --

    M app/assets/javascripts/login.js (1)
    M app/controllers/sessions_controller.rb (2)
    M app/views/sessions/new.html.erb (16)
    M config/locales/en.yml (3)
    A test/system/user_login_test.rb (68)
    M test/system/user_signup_test.rb (14)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5957.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5957.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5957
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5957 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250426/76277999/attachment.htm>


More information about the rails-dev mailing list