search

commaai / connect

comma connect PWA
https://connect.comma.ai/
MIT License
110 stars 72 forks source link

connect links outside date range do not load #520

Closed sshane closed 2 months ago

sshane commented 3 months ago

Just shows the drive list

vishalkrishnads commented 3 months ago

How exactly can this be reproduced? I tried selecting a date range, and then pasting a new drive link in the address bar. It seems to be loading fine. The drive link was outside the date range.

adeebshihadeh commented 3 months ago

How exactly can this be reproduced? I tried selecting a date range, and then pasting a new drive link in the address bar. It seems to be loading fine. The drive link was outside the date range.

To repro with the demo routes, you'll probably want to force the range to be like 1 day.

sshane commented 3 months ago

API with new input route_str to endpoint routes_segments has been deployed to staging URL: https://api.aks.comma.ai/

You'll need to add it here: https://github.com/commaai/comma-api/blob/d23b152ea64bfcd0de0937d2d525d556870820b6/src/drives.js#L15