Métriques du dépôt
- Stars
- (4 680 étoiles)
- Métriques de merge PR
- (Merge moyen 7j 4h) (20 PRs mergées en 30 j)
Description
In fixing #1868 I realized we have multiple URLs to rss feeds. It would be nice to standardize them, but we have to take special care not to break RSS URLs because people copy them into their feed readers and forget about them for a decade. If we break those URLs, our site disappears from readers, more-or-less silently depending on their UI.
Specifically, we have many urls, like /top/1w/rss, that end in /rss instead of the more familiar .rss.
This work needs a number of routing specs to confirm that rss feeds work and the old /rss all redirect to their .rss counterparts.
One odd bit of behavior popped up in #1602 that we support /page/2.rss and /top/1w/page/2.rss. (Thanks @dzwdz for help locating these.) I'm not sure these were deliberately planned, the concept is kind of puzzling (a feed starting at page 2?), and I'll have to search logs to figure out if anyone ever uses them. If there is a reason for these to exist, it would help if their specs included a comment with that reason so we don't forget in the future.
For debugging, our Routes module is handy for quick feedback on URLs: Routes.top_url('1w', format: :rss).