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 ADE-registration labels #234

Merged
merged 7 commits into from
Aug 26, 2021

Conversation

cookeac
Copy link
Collaborator

@cookeac cookeac commented Aug 11, 2021

Defines a URL scheme that supports GET, single and batch POST for members of the ADE-registration label: animals, births, deaths, arrivals, and departures.
This is modelled on the scheme for health events and once that PR is merged will only add one URL scheme file.
Helps to resolve #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 registrationURLScheme.json, which adds GET, single POST, and **batch** POST methods for animals, births, deaths, arrivals, and departures.
@cookeac cookeac requested review from ahokkonen, AndreasSchultzGEA and alamers and removed request for AndreasSchultzGEA August 18, 2021 05:31
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.

no remarks

@erwinspeybroeck erwinspeybroeck merged commit 7cf56d4 into adewg:Develop Aug 26, 2021
@cookeac cookeac deleted the cookeac-movements 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.

3 participants