[openstreetmap/openstreetmap-website] Removed unused translations in changesets namespace (PR #5223)

Anton Khorev notifications at github.com
Thu Sep 19 12:07:04 UTC 2024


@AntonKhorev commented on this pull request.



> @@ -40,9 +40,10 @@
         <% next unless comment.visible || current_user&.moderator? %>
         <li id="c<%= comment.id %>">
           <small class='text-body-secondary'>
-            <%= t comment.visible ? ".comment_by_html" : ".hidden_comment_by_html",
+            <% comment_info = comment.visible ? "comment_by_html" : "hidden_comment_by_html" %>
+            <%= t(".#{comment_info}",

Looks like once you do `t ".#{comment_info}"` i18n-tasks stops reporting `changesets.show.`-anything as missing.

You could have inserted this line with the same effect:

    <% t ".#{'shut up'}" %>

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

Message ID: <openstreetmap/openstreetmap-website/pull/5223/review/2315338676 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240919/2bc07dcd/attachment.htm>


More information about the rails-dev mailing list