TIP-Global-Health / eheza-app

Apache License 2.0
3 stars 1 forks source link

Patient Record - Antenatal - Overview Screen [12h] #98

Closed jeanette-pelizzon closed 2 years ago

jeanette-pelizzon commented 2 years ago

Issue

Adult patients will have an overview screen on their patient record that mimics case management. At the top, there will be a tab for each of the potential report types that the patient has. For example, an adult female patient will have tabs for Acute Illness, Antenatal, and Demographics.

Below the tab for Antenatal Care will be a list of all the patient's past and ongoing antenatal progress reports.

These are organized into 4 columns:

Screen Shot 2022-01-07 at 2 19 40 PM

Design

Designs for all patient record screens can be found in Figma here (under dashboards / case mgmt / progress reports)

Tasks

anvmn commented 2 years ago

@jeanette-pelizzon @isapisa

This issue is very much underestimated:

  1. This is a first time we need present summary of all pregnancies, so there's a need fot not only display, but also fetch required data.
  2. So far, antenatal report was accessed only from encounter page. Extending this to additional origin is not a trivial task.
  3. As we did with Acute Illness progress report, there're may be adjustments needed to display completed preganancy report.
adamhstewart commented 2 years ago

@anvmn according to the design suggestion the "open" label should be teal, in fact it is red:

image

Likewise "concluded" is teal and should be red.

image

adamhstewart commented 2 years ago

@anvmn can you also make a note about how "conclusion" is stored here? What about pregnancies that are not manually concluded.

anvmn commented 2 years ago

@adamhstewart

There's inconsistency in the design. At AI, we have ONGOING as red: Selection_567

Also, at progress report: Selection_568

I don't think Antenatal is special in any way.

//cc @jeanette-pelizzon

anvmn commented 2 years ago

@adamhstewart

can you also make a note about how "conclusion" is stored here? What about pregnancies that are not manually concluded.

Preganancies that are not concluded are considered open. However, there's no option to start a new pregnancy before you close previous instance.

jeanette-pelizzon commented 2 years ago

@adamhstewart i've made things that are alerts or need attention to in red to draw attention. Resolved things or things that are good now are in teal. So teal means "we're good here" and red means "hey check this out asap"

anvmn commented 2 years ago

@adamhstewart we had discussed this few weeks ago, and concluded there's no problem. Moving this to QA column.

balagan73 commented 2 years ago

https://nutrition-ihangane.pantheonsite.io/

QA Review

Found issues and questions:

  1. Issue description says about Pregnancy start: "What is the date that this pregnancy was opened in e-eheza, should be the first time seeing a CHW or Nurse for this pregnancy." The Pregnancy date doesn't reflect when the pregnancy was opened in e-heza, but when it actually started. I think that's OK. If not, I recommend also changing the column name to Pregnancy opened or something more descriptive.

Checked and approved:

1-patient-record

  1. Status column has labels with red background for Open, and with teal background for Concluded pregnancies.

  2. Other columns contain correct information.

jeanette-pelizzon commented 2 years ago

Issue description says about Pregnancy start: "What is the date that this pregnancy was opened in e-eheza, should be the first time seeing a CHW or Nurse for this pregnancy." The Pregnancy date doesn't reflect when the pregnancy was opened in e-heza, but when it actually started. I think that's OK. If not, I recommend also changing the column name to Pregnancy opened or something more descriptive.

The issue description is confusing. That's my fault. It should be the date the pregnancy actually started with the patient so it seems to be operating correctly. I will move to acceptance testing. Thanks!