-
Notifications
You must be signed in to change notification settings - Fork 232
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
Car share event types #843
Comments
I think passenger_cancellation is wrong for car share and only applies to passenger services. This should be fixed now for 2.0. Yes and trip_pause should also be added to possible events. |
Got it, so the correct list for car share is:
Done. |
This is being handled in OpenAPI in openmobilityfoundation/mds-openapi#6 |
I will make the change to the list for car share here. |
Updated car share to change references to passenger to customer instead in this commit, and updated the state machine diagram here. Ready to close issue on confirmation by @thekaveman. |
@schnuerle I think we still need this update:
|
Looking at it I actually think trip_pause should not be in Car Share. The 4 listed should work fine. I think it needs to be removed from the Trip ID Requirements list here instead: https://github.com/openmobilityfoundation/mobility-data-specification/blob/release-2.0.0/modes/car-share.md#trip-id-requirements Car Share
Micro
Robots
Passenger
All Available
|
Removed trip_pause from car share here. |
Cancellation types issue
In the
car-share
mode, under Event Types, we have the following cancellation types:In the Trip ID Requirements we see these cancellation types:
Which one is correct (or are both?) for
car-share
:Trip types issue
Trip ID Requirements lists
trip_pause
andtrip_resume
.Event Types only lists
trip_resume
, omittingtrip_pause
Presumably
trip_pause
should be an allowed Event Type here.The text was updated successfully, but these errors were encountered: