[openstreetmap/openstreetmap-website] Add pluralizers to I18n (PR #5654)

Anton Khorev notifications at github.com
Sat Mar 1 15:34:49 UTC 2025


@AntonKhorev commented on this pull request.



> @@ -1,6 +1,7 @@
 <%= tag.head :data => application_data do %>
   <%= render :partial => "layouts/meta" %>
   <%= javascript_include_tag "turbo", :type => "module" %>
+  <%= javascript_include_tag "make-plural/cardinals" %>

Name collisions. They have `const a`, `const b` etc defined in the global scope. When I added `make-plural/cardinals` as `//= require ...` in `application.js` it was picking wrong pluralization rules.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5654/review/2652499201 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250301/4c9c6b9b/attachment.htm>


More information about the rails-dev mailing list