-
Notifications
You must be signed in to change notification settings - Fork 911
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removed unused translations in changesets namespace #5223
Removed unused translations in changesets namespace #5223
Conversation
app/views/changesets/show.html.erb
Outdated
@@ -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}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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'}" %>
config/i18n-tasks.yml
Outdated
@@ -132,6 +132,7 @@ ignore_unused: | |||
- 'reports.new.categories.*' # double interpolation in reports_helper | |||
- 'shared.pagination.*' | |||
- 'auth.providers.*' | |||
- 'changesets.paging_nav.*' # used by string interpolation in elements pagination |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't be required if #4872 is merged, although it uses the same effect as in https://github.com/openstreetmap/openstreetmap-website/pull/5223/files#r1766713701 to some extent.
306d398
to
3f27dc3
Compare
Thanks, I've just made following updates:
|
PR makes following changes: