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

Feed API's first version #176

Merged
merged 92 commits into from
Dec 16, 2020
Merged

Feed API's first version #176

merged 92 commits into from
Dec 16, 2020

Conversation

erwinspeybroeck
Copy link
Collaborator

This is about issue https://github.com/adewg/ICAR/issues/93

This is the first version of de feeding API's. There are still some things to be done :

  • complete the feed properties list in the location-scheme.md file
  • carefully identify the mandatory fields
  • and of course have these checked by someone knowledgeable

Added 5 feed related endpoints
corrected url for feeds
New resource for Feed
New collection of rations
New collection of feed intake events
New collection of daily feed reports
new collection of feed recommendations
enum specifying the different feed categories
enum specifying the different feed categories
forgot Type in the name ...
create a feed type scheme and identifier
create a resource decribing a feed
feed property type creation
create a scheme and id for feed property
FeedCategory missing Type and is not in the types section, but the enums section
test to find an error
create property identifier
test with documenting the fao.org scheme
other way of documenting the schemes for feed type and feed property
added the fao.org scheme and a list of the feed types included in that scheme
@thomasd-gea
Copy link
Contributor

We might need to check which fields are required on the resources. I marked some of them, but we should recheck.

@erwinspeybroeck erwinspeybroeck merged commit e84a7c4 into adewg:Develop Dec 16, 2020
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