uwrit / leaf

Leaf Clinical Data Explorer
https://www.youtube.com/watch?v=ZuKKC7B8mHI
Other
88 stars 47 forks source link

Request: disable viewing some demographics from Patient List #471

Open jnothman opened 2 years ago

jnothman commented 2 years ago

In some applications it is appropriate to only display a subset of the core demographic fields (and yet perhaps some additional demographic fields too), and it might be appropriate to hide fields like:

Is it possible to make a reduction in demographic fields configurable?

ndobb commented 2 years ago

Definitely. This will have some impact on the Visualization screen, but I think we can add more granular configuration for the out-of-the-box visualizations to disable those so the web client knows not to expect certain fields.

@jnothman from an admin usability perspective, would you imagine this being easiest to configure by (1) simply omitting the fields from the Basic Demographics query and have Leaf detect this, or (2) via new options in the appsettings.json file?

(1) Has the advantage that it's likely just easier for admins. (2) Has the advantage that we could easily detect misconfiguration at startup and log a warning - e.g., if all Visualizations were enabled, yet the additional demographic fields above were not marked as omitted in appsettings.

Thoughts?

Beyond the visualization considerations, I don't think implementation would be particularly complicated.

jnothman commented 2 years ago

I'd say the main disadvantage of using appsettings.json is visibility. TBH, I'd also be fine with the patient list page having UI ability to hide some columns!