[openstreetmap/openstreetmap-website] Replace resent confirmation link with button (PR #5447)
Anton Khorev
notifications at github.com
Sun Dec 29 03:19:04 UTC 2024
Includes #5446
There's an issue that the *resend confirmation email* link reads "click here" #5431. I think there shouldn't be any link at all, this should be a button because it does something, not just loads a page. And the page shouldn't be accessible by GET, the button should trigger POST. When you accept the confirmation link, there's also a button which posts, but its hidden and autoclicked by javascript. The current resend link+GET existed since https://github.com/openstreetmap/openstreetmap-website/commit/bddf524c888c7364119a673e9b6c0518b0c6f93d.
Fixes #5431 by not having a link, and not writing "click here" on a button.

You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/5447
-- Commit Summary --
* Add within_content_body system test helper
* Test signup with confirmation email
* Move code that runs after user save from save_new_user to create method
* Replace resent confirmation link with button
-- File Changes --
M app/controllers/users_controller.rb (40)
M app/views/confirmations/confirm.html.erb (6)
M config/locales/en.yml (4)
M config/routes.rb (2)
M test/application_system_test_case.rb (4)
M test/controllers/confirmations_controller_test.rb (8)
M test/system/user_signup_test.rb (88)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/5447.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5447.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5447
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5447 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241228/3ce606e1/attachment.htm>
More information about the rails-dev
mailing list