thehyve / transmart-rest-api

HTTP available tranSMART RESTful API
Other
6 stars 12 forks source link

Features/cherkasy 58+59 implement concept related rest calls #6

Closed alchimystic closed 10 years ago

alchimystic commented 10 years ago

Both CHERKASY-58 and CHERKASY-59 are included Most of the implementation code was last week pushed directly into the CHERKASY-implement-api-calls

cataphract commented 10 years ago

Can you see how this is handling the study's root concept? Will it respond to /studies/study1/concepts/ ? I guess not, because that should instead give a list of concepts. If you see the self link for the ontology term serializer, I assumed that concept would be represented as /studies/study1/concepts/ROOT, which is a bit hacky as it will require special treatment in the places where concepts are loaded from ids, but I don't see any other way.

alchimystic commented 10 years ago

Study root concept is now working, according to the _self links from HAL format /studies/study1/concepts/ROOT