search

lemongrabs / get-here

wayfinding webapp to help people get from manhattan to fire island, made with love for our friends at sharegurl.com.
http://ferry.sharegurl.com
MIT License
0 stars 0 forks source link

Trip not populating #19

Open rafkuhn opened 7 years ago

rafkuhn commented 7 years ago

Plugged in 8/11 @ 8pm and no route populated. Rhonda mentioned a lot of errors when she tried to use it, but also refused to be specific with what was going on / couldn't tell me what browser she was using, so I'm not sure there's real feedback in there to troubleshoot.

zane commented 7 years ago

@oldwestaction, backend is legitimately returning to results for that date and time. I'll investigate.

zane commented 7 years ago

@rafkuhn, will be hard to troubleshoot the other errors without more detail, but we'll keep our 👀 peeled.

rafkuhn commented 7 years ago

lol I know. I was like "...I need more detail - what did you search? What browser?" And she essentially responded "AGHRHHEHSIRJNRJR"

On Jun 7, 2017, at 10:09 AM, Zane Shelby notifications@github.com wrote:

@rafkuhn, will be hard to troubleshoot the other errors without more detail, but we'll keep our 👀 peeled.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

zane commented 7 years ago

@rafkuhn, this is happening because Google Maps is legitimately returning no results for that date / time. What would the desired behavior in this kind of situation be? I could imagine changing the error message.

rafkuhn commented 7 years ago

The schedule is currently populating through 8/2. This is only A problem if ppl are booking flights for the summer way ahead of time so we should probably think about a workaround for when we add JFK. Maybe this is a 60-day in advance thing?

On Jun 7, 2017, at 10:38 AM, Zane Shelby notifications@github.com wrote:

@rafkuhn, this is happening because Google Maps is legitimately returning no results for that date / time. What would the desired behavior in this kind of situation be? I could imagine changing the error message.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

zane commented 7 years ago

@rafkuhn Could be! In which case one option would be to change the front-end behavior to disallow dates that far out.

rafkuhn commented 7 years ago

That would be a good solution for now...but lets see if the mta schedule extends tomorrow. It's not quite 60 calendar days (that would be August 6) so I'm a bit confused.

On Jun 7, 2017, at 10:46 AM, Zane Shelby notifications@github.com wrote:

@rafkuhn Could be! In which case one option would be to change the front-end behavior to disallow dates that far out.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.