girikuncoro/cageur

CSV import for Cageur API

Closed this issue · 2 comments

girikuncoro commented

Follow up for #127 , currently we can import CSV for patient info, the remaining parts:

  • template
  • disease group
  • patient disease group (this will be tricky, we should discuss more)
tororudy commented

Hi @girikuncoro, so sorry for such a long delayed work..

I'd like to discuss more regarding import for patient disease group. This will involve two api call to patient and patient disease group am I right? and for batch import, it will require iteration for every row in csv file?

I'm imagining these step in my mind, CMIW, each iteration involving:

  1. 'GET' to patient api for getting patient id since there is no way that id is in the csv, or is it possible to provide id in csv?
  2. Based on previous call to patient api, we call 'POST' to disease group api

How do you think about the above approach? I'd like to hear your comment. Thanks.

girikuncoro commented

@tororudy sorry I missed this, your approach in #141 is correct, we can move on. You only need 1 api call to /patient-disease-group assuming we know the patient id and disease group id.