Skip to content
This repository has been archived by the owner on Jun 20, 2022. It is now read-only.

Test result comment not shown #395

Open
lordpomeroy opened this issue May 21, 2020 · 1 comment · May be fixed by #415
Open

Test result comment not shown #395

lordpomeroy opened this issue May 21, 2020 · 1 comment · May be fixed by #415
Assignees
Labels
f-patient-details Issues related to the patient details page help wanted Extra attention is needed part-backend Changes to the backend structure part-client Changes to frontend structure

Comments

@lordpomeroy
Copy link
Contributor

Issue:
The test result comment that can be added is not shown in the patient details.

Steps to reproduce:
Add a test result to a patient and add a comment.
Open the patient's detail page.

Desired Solution:
The comment should be saved(?) and shown.

@lordpomeroy lordpomeroy added f-patient-details Issues related to the patient details page part-client Changes to frontend structure labels May 21, 2020
@jabrandes jabrandes self-assigned this May 24, 2020
@jabrandes jabrandes added help wanted Extra attention is needed part-backend Changes to the backend structure labels May 24, 2020
@jabrandes
Copy link
Collaborator

jabrandes commented May 24, 2020

@lordpomeroy: Indeed, the comment field was not saved in the backend. I constructed a fix for that, but I am not sure whether the behaviour I coded is desired.

There are two types of comments that can be entered for tests. The first is entered by the testing site on probe association, the second by the lab performing the test. The question that arose to me is whether both types of comments need to be stored separately such that the latter is not overriding the former. I would greatly appreciate other opinions on that specific issue.

@jabrandes jabrandes linked a pull request May 24, 2020 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
f-patient-details Issues related to the patient details page help wanted Extra attention is needed part-backend Changes to the backend structure part-client Changes to frontend structure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants