otpGraphQL: remove supportedTransitModes field

[1Maxnet1]

I found that in

transport-apis/data/de/ulm-otp.json

Line 19 in 236d88d

"supportedTransitModes": [ "TRANSIT", "CARPOOL" ]

and

transport-apis/data/de/stadtnavi-otp.json

Line 19 in 236d88d

"supportedTransitModes": [ "TRANSIT", "CARPOOL" ]

an option called "supportedTransitModes" is used. However it is not mentioned in the protocol specific options: https://github.com/public-transport/transport-apis#open-trip-planner-with-graphql

So imho it would make sense to add it there and maybe link to the possible values (e.g. this https://docs.opentripplanner.org/en/dev-2.x/RoutingModes/ if I'm not mistaken).

If you agree I could make a little PR adding the documentation

[1Maxnet1]

Actually I would say, that the supportedTransitModes can be retrieved from the api (named travelOptions see e.g. https://api.stadtnavi.de/routing/v1/router/ and http://dev.opentripplanner.org/apidoc/1.0.0/json_TravelOption.html) so we do not need to have them in the configuration file at all. As removing them would be a breaking change, I would suggest to document the option anyway and mention that they can be retrieved from the api, so the option shouldn't be used, so that it can be removed in a new major release of this repo?

However I would appreciate some thoughts on that :)

[derhuerst]

I agree that it can be removed as a breaking change.