Open multun opened 1 year ago
The energy_source field was supposed to be used soon. I would like to discuss it before finding out that there's an issue asking to remove it...
@flomonster is this fixed by the changes on the schemas ?
@flomonster is this fixed by the changes on the schemas ?
I'm not sure which changes you're talking about. But no.
We need to discuss the roadmap for rolling stock modelization.
Description and goal
Over the time, a couple of irrelevant fields creeped into the rolling stock RailJSON schema:
version
was added in this change. It's meant to be incremented every time the rolling stock is edited. It has no business there, it's supposed to be internal to editoast, and its addition is not sound: if the version counter is supposed to be incremented when a rolling stock with a given ID changes, this field cannot meaningfully be read from disk. Clients also have no business knowing the current version.energy_sources
was added in this change. This field is not read anywhere, and we don't know when it's going to be used. Sure, the design of this API is pretty good, and we're probably going to use it at some point, but we should have added the field as we needed it:Both of these changes introduced practical issues.
Acceptance criteria
version
field from the RailJSON schemaenergy_sources
field from the RailJSON schema