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

Symptoms: Building depth in consultation #3618

Closed
Tracked by #7705
aparnacoronasafe opened this issue Sep 20, 2022 · 17 comments · Fixed by #7874
Closed
Tracked by #7705

Symptoms: Building depth in consultation #3618

aparnacoronasafe opened this issue Sep 20, 2022 · 17 comments · Fixed by #7874

Comments

@aparnacoronasafe
Copy link
Member

When a new consultation is filed (https://care.coronasafe.in/facility/c92cc4df-0fea-4d37-9c24-1c9f07bfac23/patient/b144585a-bf1b-4000-8c3f-00fc859b52fc/consultation) The first field in the form is to record symptoms. Multiple symptoms can be recorded. However, the "date on onset of symptoms" field only appears as a common field for all symptoms.

Change suggested
When multiple symptoms are selected from the dropdown, "date of onset of symptom" must be taken individually for each of the symtoms. It is okey to group a few symptoms in giving the date.

Suggested format
Symptoms must be a small section within a consultation in the following design
Screenshot 2022-09-20 at 12 58 41 PM

  • There must be a checkbox to mention if the patient is asymptomatic. If selected, the rest of the fields must not appear.
  • Otherwise, the user must have the option to enter a date on onset - and mention symptoms against it. The symptoms will appear in the dropdown with multi-select option.
  • The user should be able to add more symptoms wherein another row of date and symptoms dropdown will appear.

Visualising saved data
Symptoms must appear as they appear currently on the patient dashboard (https://care.coronasafe.in/facility/015ad911-39b7-4104-b304-bf8178c33830/patient/aefb147a-eb09-4dd0-8390-e2cd5a05f326/consultation/c5e82e5f-2bab-4cab-9239-307770792bce) as a card. However, it must appear as a timeline with dates, in chronological order and the symptoms against it.

Updating symptoms
The symptoms will the updated by the nurses via the Log-Update form. Such updated symptoms must also appear in the patient dashboard, along with the respective date that the log update is filed on.

@rithviknishad
Copy link
Member

rithviknishad commented Sep 20, 2022

I'll do the front end modification's for this along with #3594

@stale
Copy link

stale bot commented Oct 9, 2022

Hi, This pr/issue has been automatically marked as stale because it has not had any recent activity. It will be closed if no further activity occurs for 7 more days. Thank you for your contributions.

@stale stale bot added the stale label Oct 9, 2022
@stale stale bot removed the stale label Oct 9, 2022
@rithviknishad rithviknishad added the waiting for related PR a co-related detail PR is under construction label Oct 9, 2022
@rithviknishad rithviknishad removed their assignment Jul 19, 2023
@rithviknishad
Copy link
Member

@aparnacoronasafe is this still required?

@Ashesh3
Copy link
Member

Ashesh3 commented Dec 26, 2023

@aparnacoronasafe could you confirm if this is still required?

@hrit2773
Copy link
Contributor

hrit2773 commented Apr 5, 2024

@rithviknishad assign me the related PR

@rithviknishad
Copy link
Member

You can create a fresh PR @hrit2773

@hrit2773
Copy link
Contributor

@rithviknishad @aparnacoronasafe mark this issue into discussion. The point of discussion would be....adding other symptoms fields for every onset date when the user enters OTHERS option there should be a separate field appearing asking the user to enter the details about other symptoms...I feel this update is required because there are multiple symptoms select fields for every onset date and this means user can enter OTHERS option multiple times.

@hrit2773
Copy link
Contributor

hrit2773 commented Apr 24, 2024

@rithviknishad @vigneshhari @Ashesh3 @sainak otherwise if the form gets complex we can remove the OTHERS option from symptom Choices and lets add an optional field of other symptoms with one onset date separately this would reduce the complexity also.

@rithviknishad
Copy link
Member

rithviknishad commented Apr 27, 2024

@hrit2773

We've discussed with @aparnacoronasafe and the following things need to be done:

  1. The JSON field should be converted to a dedicated table. The table should have the following fields: symptom (varchar with choices); other symptom (varchar, nullable); onset_date (date field); cure_date (date field); consultation (foreign key -> PatientConsultation);

  2. Through the daily rounds/log update form, we should be able to mark existing symptoms as cured or add a new symptom.

  3. Ensure existing data is migrated. (JSON to Table)

  4. When the user selects "Others" from the symptoms dropdown, a dialog popup should appear for the user to type the other symptom as text, once confirmed, that text symptom can be shown in the inline view for the symptoms select field.

cc: @aparnacoronasafe @sainak

@hrit2773
Copy link
Contributor

hrit2773 commented Apr 29, 2024

@rithviknishad the new table would contain symptoms field which is a field of multiple symptoms right and not just one symptom so the cure date would also be for multiple symptoms together right because from frontend we are selecting multiple symptoms for one onset date. can you just confirm

@hrit2773
Copy link
Contributor

hrit2773 commented May 7, 2024

@rithviknishad @aparnacoronasafe something like this? from implies the onset date and if cure date exists then the string would be " from {onset date} to {cure date} " in this case as there is no cure date so "to {cure date}" is not visible and the symptom not having a cure date is marked in red color whereas if the symptom is cured then its marked with green just changed the chip variant in frontend
Screenshot 2024-05-07 190910

@hrit2773
Copy link
Contributor

hrit2773 commented May 8, 2024

@rithviknishad @aparnacoronasafe can you also provide clarity on how to update these symptoms from both edit consultation details and log update. In edit consultation page can we add more symptoms.

@hrit2773
Copy link
Contributor

hrit2773 commented May 9, 2024

@rithviknishad for linking the symptoms with the daily rounds we also need to add a daily round foreign key field in the ConsultationSymptom model because to update log we have to get access to the symptoms which were part of that particular log update.
Screenshot 2024-05-09 222549

@hrit2773
Copy link
Contributor

hrit2773 commented May 9, 2024

@rithviknishad so in total we will have 2 foreign key fields one is consultation and other one daily round

@hrit2773
Copy link
Contributor

hrit2773 commented May 11, 2024

@rithviknishad this is the design to mark the symptoms as cured which is the symptoms status and is in the log update daily rounds form. Can you plz acknowledge it any tell me if any improvements needed in the design
Screenshot 2024-05-11 221011

@rithviknishad
Copy link
Member

rithviknishad commented May 17, 2024

@hrit2773

The backend looks almost good 😄. @sainak Will be making some minor changes to get it ready for merge.

As for the front end, many changes are required, as the code is not maintainable and the UI isn't satisfactory. Since another high-priority PR #7839 requires this feature, I'll do the front-end changes for coronasafe/care#2095 in #7839.

@hrit2773
Copy link
Contributor

@rithviknishad good by the way I was just trying to do my own design stuff. just the rough kind of UI making enhancements will be good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment