[openstreetmap/openstreetmap-website] Fix social link form resubmission after removing invalid link (PR #6945)

Matt Van Horn notifications at github.com
Sun Mar 29 06:28:23 UTC 2026


@mvanhorn commented on this pull request.



> @@ -21,6 +21,7 @@ $(function () {
       if (destroyCheckbox) {
         destroyCheckbox.checked = true;
         row.addClass("d-none");
+        row.find("input[type='text']").removeAttr("required");

You're right, the same issue applies on page load when pre-destroyed rows are re-hidden. Added the `removeAttr` there too in 8e8f42c.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6945?email_source=notifications&email_token=AAK2OLNHGCHIBAID4YAUMGT4TC7APA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBSGY2DENRZHEYKM4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOS6XA4S7OJSXM2LFO5PW433UNFTGSY3BORUW63TTL5RWY2LDNM#discussion_r3005785823
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6945/review/4026426990 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260328/065538c8/attachment.htm>


More information about the rails-dev mailing list