remove redundancies, raise question

Author: marie-x

data-types.md

@@ -218,9 +218,6 @@ A Trip is defined by the following structure:
 | `end_location`           | [Point](point)  | Required               | Location of the end of the trip. See also [Stop-based Geographic Data][stop-based-geo]. |
 | `publication_time`       | [Timestamp][ts] | Optional               | Date/time that trip became available through the trips endpoint |
 | `reservation_attributes` | [Reservation](#reservation-data) | Required if available | Reservation details, if a reservation initiated this trip
-| `dispatch_time`          | [Timestamp][ts] | Conditionally Required | Time the vehicle was dispatched to the customer (required if trip was dispatched) |
-| `parking_verification_url` | String        | Optional               | A URL to a photo (or other evidence) of proper vehicle parking |
-| `cancellation_reason`    | Enum            | Conditionally Required | The reason why a *driver* cancelled a reservation. (required if a driver cancelled a trip, and a `driver_cancellation` event_type was part of the trip) |
 | `accessibility_options`  | Enum[]          | Optional               | The **union** of any accessibility options requested, and used. E.g. if the passenger requests a vehicle with `wheelchair_accessible`, but doesn’t utilize the features during the trip, the trip payload will include `accessibility_options: ['wheelchair_accessible']`. See [accessibility-options][accessibility-options] |
 
 (?) Reconcile inconsistent use of `trip_`
@@ -236,13 +233,12 @@ Examples of mode-specific `trip_attributes`:
 | `dispatch_time`                 | [Timestamp][ts]                      | Conditionally Required | Time the vehicle was dispatched to the customer (required if trip was dispatched) |
 | `quoted_trip_start_time`        | [Timestamp][ts]                      | Required               | Time the trip was estimated or scheduled to start, that was provided to the passenger |
 | `requested_trip_start_location` | [Point](point) | Conditionally Required | Location where the customer requested the trip to start (required if this is within jurisdictional boundaries) |
-| `reservation_method`            | Enum                           | Required               | Way the customer created their reservation, see [reservation-method](#reservation-method) |
-| `reservation_time`              | [Timestamp][ts]                      | Required               | Time the customer *requested* a reservation |
-| `reservation_type`              | Enum                           | Required               | Type of reservation, see [reservation-type](#reservation-type) |
 | `cancellation_reason`           | string                         | Conditionally Required | The reason why a *driver* cancelled a reservation. (required if a driver cancelled a trip, and a `driver_cancellation` event_type was part of the trip) |
 | `accessibility_options`         | Enum[]                         | Optional               | The **union** of any accessibility options requested, and used. E.g. if the passenger requests a vehicle with `wheelchair_accessible`, but doesn’t utilize the features during the trip, the trip payload will include `accessibility_options: ['wheelchair_accessible']`. See [accessibility-options][accessibility-options] |
 | `parking_verification_url` | String | Optional | A URL to a photo (or other evidence) of proper vehicle parking |
 
+(?) Do I have the right partitioning between trip attributes and reservation attributes?
+
 [Top][toc]
 
 [agency]: /agency/README.md