search

mskcc / LimsRest

the restful service used by the IGO LIMS
2 stars 2 forks source link

Refactoring metadata #183

Closed DavidStreid closed 4 years ago

DavidStreid commented 4 years ago

Simplifying request - moving extra fields to metadata

BEFORE

{
   "request":{
      "summary":{ ... },
      "metaData":{
         "ProjectName":"FIELD NOT IN ILABS",
         "sourceProjects":[
            "07973_CE",
            "07973_CD",
            "07973_CC",
            "07973_BQ",
            "07973_CI",
            "07973_CJ"
         ],
         "QcAccessEmails":"",
         "LaboratoryHead":"Frederic Geissmann",
         "RequestName":"HemePACT_v4",
         "TATFromInProcessing":"",
         "Investigator":"Rocio Vicario",
         "ProjectManager":"Selcuklu, S. Duygu",
         "ReceivedDate":1559675432000,
         "TATFromReceiving":"",
         "GroupLeader":"Kety Huberman",
         "LabHeadEmail":"geissmaf@mskcc.org",
         "DataAccessEmails":""
      },
      "requestId":"07973_CQ",
      "bankedSampleId":"IGO-014019",
      "stages":[...],
      "samples":[...]
   },
   "requestId":"07973_CQ",
   "serviceId":"IGO-014019"
}

AFTER

{
   "summary":{ ... },
   "metaData":{
      "ProjectName":"FIELD NOT IN ILABS",
      "sourceProjects":[
         "07973_CE",
         "07973_CD",
         "07973_CC",
         "07973_BQ",
         "07973_CI",
         "07973_CJ"
      ],
      "QcAccessEmails":"",
      "serviceId":"IGO-014019",
      "LaboratoryHead":"Frederic Geissmann",
      "RequestName":"HemePACT_v4",
      "TATFromInProcessing":"",
      "Investigator":"Rocio Vicario",
      "ProjectManager":"Selcuklu, S. Duygu",
      "ReceivedDate":1559675432000,
      "TATFromReceiving":"",
      "GroupLeader":"Kety Huberman",
      "requestId":"07973_CQ",
      "LabHeadEmail":"geissmaf@mskcc.org",
      "DataAccessEmails":""
   },
   "requestId":"07973_CQ",
   "stages":[ ... ],
   "samples":[ ... ]
}

Request: https://tango.mskcc.org:8443/LimsRest/getRequestTracking?request=07973_CQ

mskbot commented 4 years ago

Can one of the admins verify this patch?