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.

Sign up for GitHub

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

Jump to bottom

Don't include patient database id in rest api output && implement rest api version number #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Don't include patient database id in rest api output && implement rest api version number #23

wants to merge 2 commits into from

Commits on Jun 13, 2016

  1. Don't include the database id when exporting patients 

    The internal database id should not be used outside of the Transmart server, the trial id (inTrialId) should be used for that. The database id
is not useful anyway since it is not exported in other places so it cannot be correlated.

The accompanying update in RInterface is in the patient_sets branch.
    @JanKanis
    JanKanis committed Jun 13, 2016
    Configuration menu
    Copy the full SHA
    d705db6 View commit details
    Browse the repository at this point in the history

  2. create /transmart-rest-api-version 

    rest api needs a version number so that clients can fail gracefully if they are not compatible.
There is at the moment no intention to have the server or R client support multiple versions at the same time.
    @JanKanis
    JanKanis committed Jun 13, 2016
    Configuration menu
    Copy the full SHA
    ec07437 View commit details
    Browse the repository at this point in the history