Closed mark-mockett closed 2 years ago
I may have missed a meeting on this because the last worker's presence meeting I attended there was not agreement on implementing it this way nor was the discussion in the issue or in person resolved. Not all comments in the issue have been addressed. I would strongly advocate for more discussion before the changes are proposed, as it is much harder to go back and change things than to do it right the first time.
Right now it feels like mostly this just adds a lot of complexity that most members (from the meetings I attended) do not care about.
(Updated 10/1 to reflect recent revisions)
This PR addresses #182
Summary of changes
worker_presence
to theRoadEvent
object with data typeWorkerPresence
objectRoadEvent
propertyworkers_present
WorkerPresence
object with the following properties:are_workers_present
– Booleansource
– WorkerPresenceSource enumerated typeworker_presence_last_confirmed_date
– string; datetimeconfidence
– WorkerPresenceConfidence enumerated typedefinition
– array ofWorkerPresenceDefinition
typeWorkerPresenceSource
WorkerPresenceDefinition
WorkerPresenceConfidence
workers_present
in the examples to use newworker_presence
property