<p></p>
<blockquote>
<p dir="auto">In <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="378228775" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/2050" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/2050/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/2050">#2050</a> (specifically <a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/openstreetmap/openstreetmap-website/commit/4b0d56f7e15d2929c973265e3b545e69273b8cb5/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/commit/4b0d56f7e15d2929c973265e3b545e69273b8cb5"><tt>4b0d56f</tt></a>) I renamed the comments_feed (which is what was used when the code was in changesets_controller) to the index method, since that's what most closely aligns with what this doing i.e. showing a list of objects. So I'd like to keep this as a method named index.</p>
</blockquote>
<p dir="auto">And I do the opposite thing in <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1896736892" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/4248" data-hovercard-type="pull_request" data-hovercard-url="/openstreetmap/openstreetmap-website/pull/4248/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/pull/4248">#4248</a> because I need the actual index path mapped to the actual index method.</p>
<blockquote>
<p dir="auto">I realise that we still have feed methods elsewhere, but those are targets for refactoring too. There's a difficulty when there are already index methods that do different things (e.g. different lists of objects, or search filters, or other things, so in those cases it can't be as simple as just a respond_to switch) but in those cases I would prefer to see e.g. FooController::index and Foo::FeedController::index or similar, i.e. the method name is still index.</p>
</blockquote>
<p dir="auto">Then that has to be done in all cases where feeds have their own paths.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/4587#issuecomment-2009950904">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLLQDHK6DMVTJ23KDHDYZGYGFAVCNFSM6AAAAABE2UEXAWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBZHE2TAOJQGQ">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLLSJOHYZVX7TLQCTDTYZGYGFA5CNFSM6AAAAABE2UEXAWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTXZVVLQ.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/4587/c2009950904</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/4587#issuecomment-2009950904",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/4587#issuecomment-2009950904",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>