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.

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

Request: disable viewing some demographics from Patient List #471

Open
jnothman opened this issue Oct 27, 2021 · 2 comments
Open

Request: disable viewing some demographics from Patient List #471

jnothman opened this issue Oct 27, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@jnothman
Copy link
Contributor

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:

  • Patient Of
  • Address Postal Code
  • Address State
  • Language
  • Religion

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

@ndobb ndobb self-assigned this Oct 28, 2021
@ndobb ndobb added the enhancement New feature or request label Oct 28, 2021
@ndobb
Copy link
Member

ndobb commented Oct 28, 2021

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
Copy link
Contributor Author

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants