-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Blacklisting API #142
Add Blacklisting API #142
Conversation
nilspenzel
commented
Sep 5, 2024
Enables fast checking of some necessary conditions for new the viability of new Tours.
639216a
to
90785a6
Compare
return json( | ||
{ | ||
body: JSON.stringify( | ||
getViableBusStops( | ||
parameters.userChosen, | ||
parameters.busStops, | ||
parameters.startFixed, | ||
parameters.capacities | ||
) | ||
) | ||
}, | ||
{ status: 200 } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return json( | |
{ | |
body: JSON.stringify( | |
getViableBusStops( | |
parameters.userChosen, | |
parameters.busStops, | |
parameters.startFixed, | |
parameters.capacities | |
) | |
) | |
}, | |
{ status: 200 } | |
return json( | |
getViableBusStops( | |
parameters.userChosen, | |
parameters.busStops, | |
parameters.startFixed, | |
parameters.capacities | |
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ginge das auch so?
startFixed ? t : new Date(t.getTime() - MAX_PASSENGER_WAITING_TIME_DROPOFF), | ||
startFixed ? new Date(t.getTime() + MAX_PASSENGER_WAITING_TIME_PICKUP) : t | ||
); | ||
new Date(times[j]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
import type { Capacities } from './capacities'; | ||
import type { Coordinates } from './location'; | ||
|
||
export type BookingRequestParameters = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BlacklistingRequestParameters ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich gehe aktuell davon aus, dass das Whiteling Parameter nach dem selben Schema Parameter annehmen wird.
Daher würde ich erstmal bei der allgemeineren Version bleiben.