project-husky / husky

Health Usability Key
https://project-husky.github.io/husky/
Eclipse Public License 1.0
17 stars 8 forks source link

CH-EMED-EPR: HtmlNarrativeGenerator add check to include or not a statement on the PDF #91

Closed dvribeira closed 1 month ago

dvribeira commented 1 year ago

With the possibility of requesting PMLC documents with non-active treatments, the generator might receive a list of medication statements for active and non-active treatments. The generator should ignore non-active medication statements.

qligier commented 1 year ago

Wouldn't it be weird to allow requesting non-active treatments for a PDF, but omit them in the actual document? I would either refuse that particular request in the PMP, or include them in the PDF.

dvribeira commented 1 year ago

It is indeed an open question. The only use case we have at the moment is the need to retrieve a structured consolidated view of all the treatments to fulfill the functional requirements of the Portail Patient and this use case does not rely on the PDF. For this reason we thought it would be enough (at least for now) to keep the PDF as what it is, the card of active treatments, whether the bundle includes entries for non active treatments or not.

Since indeed the PDF is sent as the original representation of the document, one could argue that it should include all the treatments in the bundle. In this case, the alternative would be to add another table to the PDF for non-active treatments.

dvribeira commented 1 month ago

Since the behaviour described by the issue was implemented and merged almost a year ago, I think it is sound to close this ticket. Should the implemented behaviour be reviewed it is better to create a new issue and link to this one.