Introduce milkURLScheme.json #235
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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 registrationURLScheme.json, which adds GET, single POST, and **batch** POST methods for animals, births, deaths, arrivals, and departures.
Moved milking related observations and resources into an OpenAPI file for the ADE-milk label. Covers milking visits, test days and test day results, lactation status observations, and summaries for daily milking averages and lactations. Resolves adewg#154
Corrected title and description for the scheme, and parameters for GET lactations.
cookeac
requested review from
ahokkonen,
alamers and
AndreasSchultzGEA
and removed request for
AndreasSchultzGEA
In response to comments to adewg#238
ahokkonen
approved these changes
Aug 26, 2021
erwinspeybroeck
approved these changes
Aug 26, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Separate URL scheme for ADE-1.2-milk messages, including GET, single POST, and batch POST.
I have not created POST API specs for some messages such as daily averages and lactations that are effectively summary or derived data. See if you agree.
Along with other PRs, this resolves #154