[openstreetmap/openstreetmap-website] Remove whitespace on signup fields #4471 (PR #4501)
Andy Allan
notifications at github.com
Wed Jan 31 14:53:55 UTC 2024
@gravitystorm requested changes on this pull request.
Hi @birante , thank you for this pull request.
First, I think there is a misunderstanding about the problem described in #4501. The issue described is for the login page (not the signup page), and it is for the situation where someone accidentally includes leading/trailing whitespace when trying to sign in.
Secondly, a PR to fix this issue will need tests, to prove that it works, and to prevent any future breakage if we ever refactor the code.
Finally, I would be interested to see if there is a rubygem that is already written to tackle this problem, so that we can easily apply the same solution to other forms on the website. For example, we could use something like https://github.com/holli/auto_strip_attributes
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4501#pullrequestreview-1853941345
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4501/review/1853941345 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240131/1fde2972/attachment.htm>
More information about the rails-dev
mailing list