@nertc approved this pull request.

Code works properly and solves the issue raised in the comment mentioned in the 
description.

There is also a different idea to solve the problem 
(https://github.com/openstreetmap/openstreetmap-website/pull/5449#issuecomment-2578415862),
 but this PR has other benefits too, like moving errors from `email` to 
`new_email` properly and optimizing `count.zero` case.



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

Message ID: 
<openstreetmap/openstreetmap-website/pull/5453/review/2546220...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to