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

Additional data for IcarMovementArrivalEventResource #431

Merged
merged 3 commits into from
Feb 1, 2024

Conversation

AndreasSchultzGEA
Copy link
Collaborator

In our HM-System and at least in one of our partners, there is the request to enter additional information about an arriving animal which has taken place bevor the ArrivalEventDateTime.

Copy link
Collaborator

@erwinspeybroeck erwinspeybroeck left a comment

Choose a reason for hiding this comment

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

Andreas, I'm a bit reluctant to do so. There are other API's to get that information, and what about last lactation, last lactation index, last treatment, ....
If the fields are not mandatory and nobody expects them from us, then it is of course possible ...

@cookeac
Copy link
Collaborator

cookeac commented Nov 29, 2023

I acknowledge some of this is covered by a lactation, but that this may not be immediately available to the destination farm during a movement. I'm not convinced that the solution is to add these fields to the movement event. We should look at adding them as optional status extensions to icarAnimalCoreResource, which is supplied as the animalDetail field of the movement.
I'll put this as a comment in #432 as well.

@@ -81,6 +81,22 @@
"$ref": "../enums/icarAnimalLactationStatusType.json",
"description": "Lactation status of the animal."
},
"parity": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"parity": {
"currentLactationParity": {

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@cookeac, I changed as agreed on.

@cookeac cookeac merged commit 5508719 into adewg:Develop Feb 1, 2024
@cookeac
Copy link
Collaborator

cookeac commented Feb 1, 2024

Resolves #432

@AndreasSchultzGEA AndreasSchultzGEA deleted the Arrivals_AdditionalData branch April 12, 2024 06:07
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.

IcarMovementArrivalEventResource: additional information about former events
3 participants