search

TimKJones / CribSpot

2 stars 0 forks source link

indiana university/purdue = 404 error #52

Closed jayokra closed 11 years ago

michaelstratman commented 11 years ago

Interesting, it has to do with the slash being in the name and screwing up the routing. A solution would be to go into the db and replace all instances of '/' in a schools name with a '-' or some other character. Does that sound reasonable?

michaelstratman commented 11 years ago

Screen Shot 2013-04-17 at 6 26 57 PM These are the affected universities, I changed them the /'s to -'s on my dev build and it worked. Should I update the production db?

TimKJones commented 11 years ago

Make sure the links generated by twitter and facebook share also still work. I think they should since it just replaces spaces. I think that would be the only other place. Evan, can you think of others?

On Wed, Apr 17, 2013 at 6:27 PM, Michael Stratman notifications@github.comwrote:

[image: Screen Shot 2013-04-17 at 6 26 57 PM]https://f.cloud.github.com/assets/2209717/394103/f2dc727a-a7ad-11e2-8391-8fa83315db49.png These are the affected universities, I changed them the /'s to -'s on my dev build and it worked. Should I update the production db?

— Reply to this email directly or view it on GitHubhttps://github.com/TimKJones/CribSpot/issues/52#issuecomment-16539732 .

michaelstratman commented 11 years ago

Also their already exist Universities who's name's contain -'s Umich being one of them, so by changing the name in the db to use a - instead of a / I don't foresee any issues.