[openstreetmap/openstreetmap-website] Add Turbo to replace custom JS (PR #4562)

Tom Hughes notifications at github.com
Thu Mar 21 11:04:37 UTC 2024


@tomhughes commented on this pull request.



> @@ -2,6 +2,7 @@
   <meta http-equiv="X-UA-Compatible" content="IE=edge" />
   <meta name="viewport" content="width=device-width, initial-scale=1">
   <%= javascript_include_tag "es6" unless browser.es6? %>
+  <%= javascript_include_tag "turbo", :type => "module" %>

Well we haven't used nonces before so it hasn't been an issue before now but sure that can be a separate pull request. I think I have a branch somewhere where I started on moving to the rails builtin policy stuff so I should probably resurrect that at some point...

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

Message ID: <openstreetmap/openstreetmap-website/pull/4562/review/1951821323 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240321/b6b6300a/attachment.htm>


More information about the rails-dev mailing list