Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sorting of animals #456

Closed
AndreasSchultzGEA opened this issue Apr 15, 2024 · 2 comments · Fixed by #457
Closed

Sorting of animals #456

AndreasSchultzGEA opened this issue Apr 15, 2024 · 2 comments · Fixed by #457
Assignees

Comments

@AndreasSchultzGEA
Copy link
Collaborator

AndreasSchultzGEA commented Apr 15, 2024

Animals might have to be sorted on a farm.
Sorting in general is about getting animals from one area of a farm to another.
The way in between has to be managed by the HMS.
Therefore, a sorting-API should

  • deliver the sorting-areas of a location
  • deliver the currently active sorting-command for an animal
  • allow the inactivation of a sorting-command for an animal
  • allow the setting of a new sorting-command for an animal

PR: #457

@cookeac
Copy link
Collaborator

cookeac commented Apr 18, 2024

Outcomes of the discussion 2024-04-18:

  • Consider standardising locations and areas as "sites". We could then align the Position observations to match.
  • Rename the resource (not event) to icarAnimalSortingCommandResource. This could then create a pattern for other commands in future.
  • Replace isPermanent with validFrom (which replaces startdate) and validTo. Setting validFrom with the date/time that the command was activated, and leaving validTo blank makes the command permanent until replaced.

@cookeac
Copy link
Collaborator

cookeac commented May 16, 2024

Resolved by #457

@cookeac cookeac closed this as completed May 16, 2024
@cookeac cookeac linked a pull request May 16, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants