596 and 597 id and shiftNumber for icarAnimalSortingCommandResource

[cookeac]

Add id and shiftNumber to icarAnimalSortingCommandResource:

• id allows unique identification of sorting commands. Resolves Unique identifier for icarAnimalSortingCommandResource. #596
• shiftNumber adds control within a milking (or weighing or other) shift, inside a validity period. Resolves Extend icarAnimalSortingCommandResource with milkshifts #597

[Copilot]

Pull request overview

This PR extends icarAnimalSortingCommandResource to better support command lifecycle management and shift-specific sorting behavior.

Changes:

• Add an id field intended to uniquely identify sorting commands (issue #596).
• Add a shiftNumber field intended to scope sorting within a shift inside a validity period (issue #597).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Harry-UA]

shiftNumbers array is optional but the minimum size is one according to the documentation. Shouldn't this be zero?

[mtech-ashelepaev]

C# code generation was tested, and the behavior is expected for the default openapi-generator-cli.
The new id property is generated as nullable, while the array property remains non-nullable. Adding "nullable": true in the schema does not change this behavior, which is a known limitation. Also, the "minimum": 1 constraint has no effect on the generated C# code.