Fixes #34, where a removed '/' suddenly causes 404 errors

umts / BusInfoBoard

A place to view bus arrival information from Avail's JSON feed

Apache License 2.0

15 stars 3 forks source link

Fixes #34, where a removed '/' suddenly causes 404 errors #35

Closed akaplo closed 8 years ago

akaplo commented 8 years ago

This will close #34.
The call url.split('/').slice(0,-1).join("/") + query_string + "&mobile=true&stops=" + stops.join("+"); was removing a '/' from the final URL that seemed to be causing problems. I managed to test it on my machine and this seems to fix it.

I removed the split('/') from the call. That's it.

akaplo commented 8 years ago

Seems to have fixed itself. Must've been something weird on Github's end.

werebus commented 8 years ago

I would be interested in still addressing this issue, but I don't think this fix is quite correct.

werebus commented 8 years ago

Taken care of with #36