GitLab

• Make existing action a oneOf with action.action_type as the discriminator
• Existing action definition becomes action_move with action.action_type=move
• Create a action_waypoint_list with action.action_type=waypoint_list and add to oneOf
  • Properties
    • waypoints array of GeoJSON:Point
• Make existing action schema accept either separate latitude/longitude fields or geojson.Point
• Update test fixtures
• Update tests

I've changed the scope of this for now at least.

Now that the other behaviour changes are being handled by the instruction set we no longer need to be constrained by all the waypoints being collated into a single action.

changed milestone to %CB-2024W35

added Partner::NOC Project::MAS-DT Status::Sprint Priority Weight::3 labels

Thinking about it. We don't actually need to do this at all. We can leave it as is (bar switching to GeoJSON) and then ORI can send multiple actions with 1 waypoint in each.

The original plan for moving away from this was to avoid specifying a mixture of waypoints and pitch/thrust changes which couldn't be easily represented on the glider. Since we're now dealing with the behaviour changes in a different message that requirement no longer exists.

added Status::In Progress label and removed Status::Sprint Priority label

created merge request !43 (merged) to address this issue

mentioned in merge request !43 (merged)

changed the description

removed Status::In Progress label

added Status::In Review label

assigned to @danjon

mentioned in issue #49 (closed)

changed milestone to %CB-2024W37

closed

mentioned in issue #68