<p>This PR refactors various links to use named paths, rather than the <code>controller => ... action => ...</code> alternative. This makes them easier to read, particularly when dealing with <code>display_name</code> parameters.</p>
<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/3293'>https://github.com/openstreetmap/openstreetmap-website/pull/3293</a></p>
<h4>Commit Summary</h4>
<ul>
<li>Use user_account_path for links to settings page</li>
<li>Use named paths for various user blocks links</li>
<li>Use named paths for diary comments path</li>
</ul>
<h4>File Changes</h4>
<ul>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/3293/files#diff-961c5d7e0310073ca00d502eda2687808102441be660a654cab29645fbcea019">app/controllers/confirmations_controller.rb</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/3293/files#diff-cfdccd0a9d5df5a43aaad2a35d36ebbe187c52ad5fdc9846fa189d04537adb6e">app/controllers/users_controller.rb</a>
(10)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/3293/files#diff-6d2ef58207c92d1533287af551a075122ba98240a861819047a2ae77b7674f0b">app/views/layouts/_header.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/3293/files#diff-72904e21fa88b5df05388f6fc78d8b4c0e8ee2c7135b6bb46b6ed5a4b50d7d7a">app/views/site/edit.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/3293/files#diff-5600ede6a40cd2ed0b592ddd5fd56e0a863da7c72dc1e80443d3ebd056b9a7e5">app/views/user_blocks/_block.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/3293/files#diff-d669c04f0a7e45151d990c7457c79db874d8d6c47e41ee6817048eeb600cc9a1">app/views/user_blocks/show.html.erb</a>
(2)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/openstreetmap/openstreetmap-website/pull/3293/files#diff-78ff736409d758722403cce31873ba803b3fb526d37398ca21caa557b54dfd95">app/views/users/show.html.erb</a>
(16)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/3293.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/3293.patch</a></li>
<li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/3293.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/3293.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/3293">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLJWZYHWYYYP527DDDTT4P5GHANCNFSM5CBV7X5A">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email">Android</a>.<img src="https://github.com/notifications/beacon/AAK2OLJUNLL6XBUOEUBWKMLT4P5GHA5CNFSM5CBV7X5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4OOE2WKA.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/3293",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3293",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>