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

URL scheme for Reproduction observations #236

Merged
merged 7 commits into from
Aug 26, 2021

Conversation

cookeac
Copy link
Collaborator

@cookeac cookeac commented Aug 12, 2021

Create a separate URL scheme for reproduction observations (label ADE-1.2-reproduction). There are 11 observations in this set, including mating, reproductive status, pregnancy checks, abortion, and parturition.
Along with other PRs, this resolves #154

Renamed icarErrorResource to icarBatchResultMessage, and added icarBatchResultSeverityType enum.
Batch POSTS now always return an array of icarBatchResult, each of which may contain an array of icarErrorWarningMessageResource.

Single POSTs if successful return the posted object (ID filled in), otherwise they return an errors array of icarErrorWarningMessageResource.

Sorry for renaming the file, but it helped to clarify its purpose.
Renamed icarErrorWarningMessage Resource to icarResponseMessageResource based on feedback to PR adewg#232
Create a separate URL scheme for reproduction observations (label ADE-1.2-reproduction). There are 11 observations in this set, including mating, reproductive status, pregnancy checks, abortion, and parturition.
Along with other commits, this resolves adewg#154
@cookeac cookeac changed the base branch from ADE-1 to Develop August 12, 2021 04:40
@cookeac cookeac requested review from ahokkonen, alamers and AndreasSchultzGEA and removed request for AndreasSchultzGEA August 18, 2021 05:34
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.

ok for me, no remarks

@erwinspeybroeck erwinspeybroeck merged commit 36d4625 into adewg:Develop Aug 26, 2021
@cookeac cookeac deleted the cookeac-reproduction branch August 27, 2021 06:11
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.

Semantics for POST method
3 participants