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 #457

Merged
merged 8 commits into from
Apr 30, 2024
Merged

Sorting of animals #457

merged 8 commits into from
Apr 30, 2024

Conversation

AndreasSchultzGEA
Copy link
Collaborator

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

@cookeac
Copy link
Collaborator

cookeac commented Apr 21, 2024

This looks good. If this is approved I will also update #430 to allow site identifiers, optionally referring to the same sites.
One final question: might someone want to query all the active animal sorting commands for a location? If so, we would need a collection and API query for this.

resources/icarAnimalSortingCommandResource.json Outdated Show resolved Hide resolved
resources/icarAnimalSortingCommandResource.json Outdated Show resolved Hide resolved
resources/icarAnimalSortingCommandResource.json Outdated Show resolved Hide resolved
@AndreasSchultzGEA
Copy link
Collaborator Author

I added an endpoint to request al known sorting-commands.
@cookeac, could you please review the changes again?
Thx!

Copy link
Collaborator

@cookeac cookeac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@cookeac cookeac merged commit 0becf0e into adewg:Develop Apr 30, 2024
@cookeac cookeac linked an issue May 16, 2024 that may be closed by this pull request
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 this pull request may close these issues.

Sorting of animals
2 participants