search

thegazelle-ad / gazelle-server

Server for front-end and editor tools of The Gazelle
MIT License
19 stars 8 forks source link

URL validation #86

Open emilgoldsmith opened 7 years ago

emilgoldsmith commented 7 years ago

Right now: https://www.thegazelle.org/issue/YoMamaSucks/BigHairyBalls/nyuad-hangover

is a valid URL. (sorry for the childish reference, just to emphasize what one could put a link out on Facebook of right now and have a laugh at The Gazelle)

lingz commented 7 years ago

Probably the best fix is if we match an article in this way, we can just redirect to the correct URL.

emilgoldsmith commented 7 years ago

Also right now putting in a url like this http://localhost:3000/issue/108/letters/bagot-4 (should've been in issue 48) will display the issue and date incorrectly, so it's not just a wrong URL it also screws up the front-end. Just another reason to fix this.