Skip to content
This repository has been archived by the owner on Jan 4, 2023. It is now read-only.

Display Ressource narrative if present #66

Open
MiskoG opened this issue Jan 27, 2022 · 2 comments · May be fixed by #68
Open

Display Ressource narrative if present #66

MiskoG opened this issue Jan 27, 2022 · 2 comments · May be fixed by #68
Assignees
@BPierrick @nriss
Labels
proj/vue-patient

Comments

@MiskoG
Copy link

MiskoG commented Jan 27, 2022 •
edited
Loading

To be explored, as @nriss and @BPierrick suggested to me the FHIR ressource narrative could be very valuable in order to better understand a ressource displayed (rather than just reading the whole payload).

As a reminder, there are 2 types of ressources in our app

  • the Patient
  • other FHIR ressources related to this Patient
@nriss
Copy link
Contributor

nriss commented Jan 27, 2022

For info: attributes placed in narrative is marked as isSummary in the ElementDefinition.

The narrative doesn't seem to be present every time, we should see HAPI params

@BPierrick
Copy link
Contributor

BPierrick commented Jan 27, 2022 •
edited
Loading

What about security then?

We're basically displaying HTML code from resource attribute string value, and this raises some security questions.

I'm not a specialist, but we should consider it if we execute this 'code' ?

I was thinking of using this package to "validate" the HTML string : https://github.com/cure53/DOMPurify

@BPierrick BPierrick linked a pull request Jan 27, 2022 that will close this issue
Open
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@BPierrick BPierrick

@nriss nriss

Labels
proj/vue-patient
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Display resource narrative arkhn/timeline-patient
3 participants
@MiskoG @BPierrick @nriss