[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