onaio / fhir-tooling

A command line utility to support FHIR Core content authoring
Other
1 stars 1 forks source link

Add an Administrative level for each Location #172

Closed dubdabasoduba closed 3 months ago

dubdabasoduba commented 3 months ago

Issue Context?

Issue Implementation details?

Issue Acceptance criteria?

Relevant Information

Sample Codes

[
  {
    "system" : "https://smartregister.org/codes/administrative-level",
    "code"   : "0"                                                   ,
    "display": "Level 0"
  },
  {
    "system" : "https://smartregister.org/codes/administrative-level",
    "code"   : "1"                                                   ,
    "display": "Level 1"
  }
]
Wambere commented 3 months ago

@dubdabasoduba thinking it might make sense to add an optional column in the csv for this value as well, such that if the value is in the csv we use it and if not we pull it from the parent location endpoint. Just being cautious of how much more time it will take to load locations if we always have to hit the parent endpoint

dubdabasoduba commented 3 months ago

@dubdabasoduba thinking it might make sense to add an optional column in the csv for this value as well, such that if the value is in the csv we use it and if not we pull it from the parent location endpoint. Just being cautious of how much more time it will take to load locations if we always have to hit the parent endpoint

@TraciebelWairimu and I had a chat about this today. @Wambere I agree with you the column on the CSV would be the easiest way to build this.

We might need to add some validation on the tools to make sure the levels are added correctly. This is meant to validate the user entry