<p>I noticed this while working on the diary entries, so now I'm rolling it out more widely. In many cases we've been avoiding using the built-in translation keys for submit buttons, and hard-coding translations based on the file the form is in (rather than based on the model for the form). This has made some form refactoring harder than it needs to be, as well as just being a bit of a waste of time. Sometimes this has been done just because of copying other places where it was done first.</p>
<p>There's ample opportunity to improve these labels in the future - too many saves and not enough updates in my opinion! But I'll leave the strings mostly unchanged for now.</p>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=1109395" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/Nikerabbit">@Nikerabbit</a> this PR contains a few translation key renames:</p>
<ul>
<li><code>diary_entries.show.save_button</code> -> <code>helpers.submit.diary_comment.create</code></li>
<li><code>messages.new.send_button</code> -> <code>helpers.submit.message.create</code></li>
<li><code>oauth_clients.new.submit</code> -> <code>helpers.submit.client_application.create</code></li>
<li><code>oauth_clients.edit.submit</code> -> <code>helpers.submit.client_application.update</code></li>
<li><code>redactions.new.submit</code> -> <code>helpers.submit.redaction.create</code></li>
<li><code>redactions.edit.submit</code> -> <code>helpers.submit.redaction.update</code></li>
<li><code>traces.new.upload_button</code> -> <code>helpers.submit.trace.create</code></li>
<li><code>traces.edit.save_button</code> -> <code>helpers.submit.trace.update</code></li>
<li><code>user_blocks.new.submit</code> -> <code>helpers.submit.user_block.create</code></li>
<li><code>user_blocks.edit.submit</code> -> <code>helpers.submit.user_block.update</code></li>
</ul>
<hr>
<h4>You can view, comment on, or merge this pull request online at:</h4>
<p> <a href='https://github.com/openstreetmap/openstreetmap-website/pull/2257'>https://github.com/openstreetmap/openstreetmap-website/pull/2257</a></p>
<h4>Commit Summary</h4>
<ul>
<li>Use built-in form sumbit translation keys for user blocks</li>
<li>Use built-in form sumbit translation keys for redactions</li>
<li>Use built-in form sumbit translation keys for messages</li>
<li>Use built-in form sumbit translation keys for diary comments</li>
<li>Use i18n for submit button for issue comments</li>
<li>Use built-in form submit translation keys for oauth client applications</li>
<li>Use built-in form submit translation keys for traces</li>
</ul>
<h4>File Changes</h4>
<ul>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-0">app/views/diary_entries/show.html.erb</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-1">app/views/issues/_comments.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-2">app/views/messages/new.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-3">app/views/oauth_clients/edit.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-4">app/views/oauth_clients/new.html.erb</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-5">app/views/redactions/edit.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-6">app/views/redactions/new.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-7">app/views/traces/edit.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-8">app/views/traces/new.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-9">app/views/user_blocks/edit.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-10">app/views/user_blocks/new.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-11">config/locales/en.yml</a>
(28)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-12">test/application_system_test_case.rb</a>
(6)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257/files#diff-13">test/system/issues_test.rb</a>
(4)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/2257.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/2257.patch</a></li>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/2257.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/2257.diff</a></li>
</ul>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2257?email_source=notifications&email_token=AAK2OLIB5GMBZSA4C6WC62LP2EZRJA5CNFSM4HXNA2A2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZEAXAQ">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLJICKPGJO6Q2CFPBODP2EZRJANCNFSM4HXNA2AQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAK2OLKVZJ5T44677BZDK3LP2EZRJA5CNFSM4HXNA2A2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZEAXAQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2257?email_source=notifications\u0026email_token=AAK2OLIB5GMBZSA4C6WC62LP2EZRJA5CNFSM4HXNA2A2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZEAXAQ",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2257?email_source=notifications\u0026email_token=AAK2OLIB5GMBZSA4C6WC62LP2EZRJA5CNFSM4HXNA2A2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZEAXAQ",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>