search

RADAR-base / RADAR-RestApi

RESTful interface to access near real-time data
Apache License 2.0
2 stars 0 forks source link

Modularisation #27

Closed fnobilia closed 7 years ago

fnobilia commented 7 years ago

Changed input parameter from userID to patientID (resolve #15 ). Sensors end-points support the Time-Frame parameter (resolve #17 ). Define the concept of Device Catalog using the configuration file device-catalog.yml (resolve #20 ).

blootsvoets commented 7 years ago

Could we use userId or subjectId instead of patientId? Since it is an ID of user in our system, but patientId implies that it is a patient, which in many research studies is not the case. In OpenMHealth header, user_id is used. subjectId would also be fine in the context of a research study, although it would be less appropriate for self-organized data gathering.

fnobilia commented 7 years ago

This was discussed during a call with the Front-end meeting a few moths ago. I see your point: userId is more general and for this reason it was initially defined as userId. @herkulano asked for this change.

herkulano commented 7 years ago

@fnobilia @blootsvoets I agree with subjectId. userId can easily be mistaken or confused with the user of the applications, either in the dashboard or the questionnaire.

//cc @kerzmaximilian

fnobilia commented 7 years ago

Changed here RADAR-CNS/RADAR-RestApi@254acef