Code base - review and improvements

[anvmn]

Tasks:

• Apply Elm review (client + server app)
• Review and improve current code

[anvmn]

@LudmilaGoichman

Please run sanity on Nutrition env for activities listed bellow. Thing to look for:

• Forms UI (not broken)
• Proper count of completed and totals tasks on form
• Ability to save form

When things look strange, compare with CHW env.

Activities list:

1. Acute Illness:
   • Physical Exam
   • Laboratory
   • Exposure
   • Treatment Review
   • Ongoing Treatment Review (at subsequent encounters)
   • Danger signs
   • Next Steps

2. Well Child:

   • Nutrition
   • Danger signs
   • Medication
   • Next Steps

3. HIV:

   • Next Steps

4. NCD:

   • Medical history
   • Next steps (initial and recurrent parts of encounters)

5. Nutrition individual:

   • Next Steps

6. Prenatal:

   • Treatment Review
   • Examination (initial and recurrent parts of encounters)
   • Next Steps (initial and recurrent parts of encounters)

7. Tuberculosis:

   • Medication
   • Next Steps

[LudmilaGoichman]

@anvmn

• Nutrition and CHW env
• Android tablet

Acute Illness: Physical Exam

Questions/Found Issues:

1. In both env. In the "Respiratory" activity, "Cough" - can be saved without selecting "More than 2 weeks" / "2 weeks or less".

   

2. I think the word "weeks" must be written in lowercase letters

3. When changing the date in nodes, errors appear. Is it okay?

4. In both env. Core Exam. Why can't select 2 symptoms in the first section? Is it okay?

5. In both env. In the first encounter in the Physical Exam task, there is Acute Findings activity, but in the subsequent encounter, this activity doesn't appear. Is it okay?

first encounter	subsequent encounter

[LudmilaGoichman]

@anvmn

• Nutrition and CHW env
• Android tablet

Acute Illness: Laboratory and Danger signs

Questions:

1. In both env. Laboratory activity: only 2 tests: Malaria and Covid (performed by a nurse). Is it okay?

2. In both env. Danger signs activity appears only in the subsequent encounter. Is it okay?

first encounter:

subsequent encounter:

[anvmn]

@LudmilaGoichman

Acute Illness: Physical Exam

1. In both env. In the "Respiratory" activity, "Cough" - can be saved without selecting "More than 2 weeks" / "2 weeks or less".
2. I think the word "weeks" must be written in lowercase letters

Opened separate issue for this - #1276.

3. When changing the date in nodes, errors appear. Is it okay?

That happens because of attempt to upgrade PHP version to 8.2. We decided not to, so the error is not relevant.

4. In both env. Core Exam. Why can't select 2 symptoms in the first section? Is it okay?
5. In both env. In the first encounter in the Physical Exam task, there is Acute Findings activity, but in the subsequent encounter, this activity doesn't appear. Is it okay?

This is expected behaviour, implemented per customer requirements.

[anvmn]

@LudmilaGoichman

Acute Illness: Laboratory and Danger signs

1. In both env. Laboratory activity: only 2 tests: Malaria and Covid (performed by a nurse). Is it okay?
2. In both env. Danger signs activity appears only in the subsequent encounter. Is it okay?

This is expected behaviour, implemented per customer requirements.

[LudmilaGoichman]

@anvmn

• Nutrition and CHW env
• Android tablet

Acute Illness:

• Exposure
• Treatment Review
• Ongoing Treatment Review (at subsequent encounters)
• Next Steps

Questions/Found Issues:

1. In both env. This phrase appears in the Exposure activity when the following symptoms are absent in the Symptoms activity:

2. In both env. I didn't find the Ongoing Treatment Review (at subsequent encounters) activity. The next steps activity is referral to hospital.

3. In both env. subsequent encounters. In the Next steps there is the Health Education activity, but there is refer to the hospital. Is it okay?

[LudmilaGoichman]

@anvmn

• Nutrition and CHW env
• Android tablet

Well Child:

• Nutrition

• Danger signs

• Medication

• Next Steps

  HIV:

• Next Steps

Questions/Found Issues:

1. Well Child. I didn't find the activity Medication
2. Well Child. First and subsequent encounters. Z-score doesn't appear for the previous measurement. Is it okay?

3. Well Child. Next Steps. Activities are different. Is it okay?

4. Well Child. Next Steps. Subsequent encounter. Why is the patient referred to the medical center? Symptoms are normal. Patient: Kuzwa Anne

5. Both env. Well Child. The "Save" button in the "Next visit" action doesn't respond for a long time. In the report, the "End encounter" button is disabled (for the first time). After reloading the page, the button was enabled. And this button and the "continue" button don't respond for a long time.

https://github.com/user-attachments/assets/4f50d644-9c32-4a33-b387-ad85d2124519

6. HIV: Next steps. First and subsequent encounters. Can't check in CHW env, there is no this encounter.

7. HIV. There is no Progress Report. Is it okay?

[anvmn]

@LudmilaGoichman

Acute Illness:

• Exposure
• Treatment Review
• Ongoing Treatment Review (at subsequent encounters)
• Next Steps

1. In both env. This phrase appears in the Exposure activity when the following symptoms are absent in the >Symptoms activity:
2. In both env. subsequent encounters. In the Next steps there is the Health Education activity, but there is >refer to the hospital. Is it okay?

Per requirements.

2. In both env. I didn't find the Ongoing Treatment Review (at subsequent encounters) activity. The next steps activity is referral to hospital.

Ongoing treatment is available at subsequent encounter, if at previous encounter medication was prescribed.

[LudmilaGoichman]

Comment https://github.com/TIP-Global-Health/eheza-app/issues/1266#issuecomment-2295163733

I checked SPV encounter:

1. the activity Medication looks well
2. the 2 and 5 points also happened
3. When in the Danger signs activity, there are symptoms, they do not appear in the report. And when the cancel pop-up about Acute Illness, in the Next steps only the next visit activity. Is it okay?

first encounter:

subsequent encounter (at the beginning):

symptoms from the Nutrition appear:

Comment https://github.com/TIP-Global-Health/eheza-app/issues/1266#issuecomment-2285704794

2. In both env. I didn't find the Ongoing Treatment Review (at subsequent encounters) activity. The next steps activity is referral to hospital.

I didn't find the Ongoing Treatment Review (at subsequent encounters). There is Treatment Review activity (as in the first encounter). is it okay?

First encounter:

subsequent encounter:

[LudmilaGoichman]

@anvmn

• Nutrition and CHW env
• Android tablet

NCD:

• Medical history
• Next steps (initial and recurrent parts of encounters)

Questions (in both env):

1. Should the patient be referred to the hospital in the Next Steps activity? When there is no this test

2. HBA1C test. Should today's date be displayed on the screen?

3. Next Steps. Should the Next Visit activity appear?

4. The Progress Report exists. Should this report is displayed when closing this encounter?

5. In the subsequent encounter, there is no the Medical history task, but there is the Outside Care activity. Is it okay?

6. Next steps. In the first and subsequent encounters, there is the medications distribution activity. Is it okay?

first encounter	subsequent encounter

7. patient: Akuzwe Adeline. Next steps. In the first and in the subsequent encounters, there is Hypertension. But the medications are given only at the subsequent encounter. Is it okay?

first encounter

subsequent encounter

8. Next Steps. Should there be Health Education for HIV?

9. Next Steps. Should the patient be referred for ARV?

10. Laboratory. Why is there a point of care and lab for blood sugar testing, but only case management for other tests?

11. patient Akuzwe Agnes. In the Next Steps, "diabetes" appears, but diagnosis is "Gestational Diabetes"

12. patient Akuzwe Agnes. In the Next Steps, there is no referral to the hospital, when tests are bad

case management:

[LudmilaGoichman]

@anvmn

• Nutrition and CHW env
• Android tablet

Nutrition individual:

• Next Steps

Questions (in both env):

1. The Progress Report exists. Should this report is displayed when closing this encounter?

2. First and subsequent encounters. Z-score doesn't appear for the previous measurement. Is it okay?

3. Nutrition env. patient Akuzwe Marie. The First encounters. One Z-score doesn't appear for the current measurement.

Nutrition env. patient Akuzwe Anne. The Subsequent encounters. One Z-score doesn't appear for the current measurement.

CHW env. patient ABIJURU Joelle. The first encounters. One Z-score doesn't appear for the current measurement.

But in other screens both Z-score appear:

[LudmilaGoichman]

@anvmn

• Nutrition and CHW env
• Android tablet

Prenatal:

• Treatment Review
• Examination (initial and recurrent parts of encounters)
• Next Steps (initial and recurrent parts of encounters)

Tuberculosis:

• Medication
• Next Steps

Questions (in both env):

1. Prenatal. In the first encounter, there is no Treatment Review activity (only the Medication activity), but it appears in the subsequent encounter. Is it okay?

the first encounter	the subsequent encounter

2. Prenatal. Why are there no risk factors listed in the report?

3. Tuberculosis. There is difference in environments

the subsequent encounter:

Nutrition	CHW

4. Tuberculosis. There is difference in environments. In Nutrition there is report, in CHW there is no report

Nutrition	CHW

[LudmilaGoichman]

@anvmn All sanity tests were successfully performed, except for the questions.

[anvmn]

@LudmilaGoichman

Well Child:

• Nutrition

• Danger signs

• Medication

• Next Steps

  HIV:

• Next Steps

Questions/Found Issues:

1. Well Child. I didn't find the activity Medication

Well child encounter is executed by CHW. There's not Medication activity there. Well child encounter is executed by nurse is called SPV, and that's where you can find that activity.

2. Well Child. First and subsequent encounters. Z-score doesn't appear for the previous measurement. Is it okay?

Yes, it's ok. Z-score displayed is only for current encounter.

3. Well Child. Next Steps. Activities are different. Is it okay?

Next steps depend vaccinations status and nutrition assessment. You had green nutrition assessment, therefore some steps were skipped.

4. Well Child. Next Steps. Subsequent encounter. Why is the patient referred to the medical center? Symptoms are normal. Patient: Kuzwa Anne

Patient is sent to HC, in case we determine that it's behind on vaccinations.

5. Both env. Well Child. The "Save" button in the "Next visit" action doesn't respond for a long time. In the report, the "End encounter" button is disabled (for the first time). After reloading the page, the button was enabled. And this button and the "continue" button don't respond for a long time.

It takes few seconds, as we navigate to progress report page, where heavy charts needs to be drawn. As for disabled "End encounter" button, I tried to reproduce, but it worked for me so far. Do you manage to reproduce it?

6. HIV: Next steps. First and subsequent encounters. Can't check in CHW env, there is no this encounter.

I see. Any concerns you had, that require this comparement?

7. HIV. There is no Progress Report. Is it okay?

No progress report for HIV was developed.

[anvmn]

@LudmilaGoichman

Comment https://github.com/TIP-Global-Health/eheza-app/issues/1266#issuecomment-2295163733

I checked SPV encounter:

2. the 2 and 5 points also happened

Answered at https://github.com/TIP-Global-Health/eheza-app/issues/1266#issuecomment-2308758977.

3. When in the Danger signs activity, there are symptoms, they do not appear in the report. And when the cancel pop-up about Acute Illness, in the Next steps only the next visit activity. Is it okay?

If CHW activity behaves in same manner, it's ok.

I didn't find the Ongoing Treatment Review (at subsequent encounters). There is Treatment Review activity (as in the first encounter). is it okay?

No Treatment review at SPV.

[anvmn]

@LudmilaGoichman

NCD:

• Medical history
• Next steps (initial and recurrent parts of encounters)

Questions (in both env)

For all points - if the behaviour is similar in both envs, and UI and functionality are not broken, it's ok. I read through all 12 points, and did not see any such problems. If I missed something, please let me know.

[anvmn]

@LudmilaGoichman

Nutrition individual:

• Next Steps

Questions (in both env):

1. The Progress Report exists. Should this report is displayed when closing this encounter?

Behaviour should be similar to CHW evn.

2. First and subsequent encounters. Z-score doesn't appear for the previous measurement. Is it okay?

Z-score is displayed for current value only.

3. Nutrition env. patient Akuzwe Marie. The First encounters. One Z-score doesn't appear for the current measurement.

Weight for height z-score is available for age of 0 to 5 years. Patients that do not shoe the score are 6 and 7 years old.

[anvmn]

@LudmilaGoichman

Prenatal:

• Treatment Review
• Examination (initial and recurrent parts of encounters)
• Next Steps (initial and recurrent parts of encounters)

Questions (in both env):

1. Prenatal. In the first encounter, there is no Treatment Review activity (only the Medication activity), but it appears in the subsequent encounter. Is it okay?

Yes, it's per requirements.

2. Prenatal. Why are there no risk factors listed in the report?

Here's the list of risk factors:

[ FactorNumberOfCSections 0
, FactorCSectionInPreviousDelivery
, FactorCSectionReason
, FactorPreviousDeliveryPeriod
, FactorSuccessiveAbortions
, FactorSuccessivePrematureDeliveries
, FactorStillbornPreviousDelivery
, FactorBabyDiedOnDayOfBirthPreviousDelivery
, FactorPartialPlacentaPreviousDelivery
, FactorSevereHemorrhagingPreviousDelivery
, FactorPreeclampsiaPreviousPregnancy
, FactorConvulsionsPreviousDelivery
, FactorConvulsionsAndUnconsciousPreviousDelivery
, FactorIncompleteCervixPreviousPregnancy
, FactorVerticalCSectionScar
, FactorGestationalDiabetesPreviousPregnancy
]

FactorVerticalCSectionScar is set at Obstetrical Exam. All others are from Obstetric History (Step 2).

[anvmn]

@LudmilaGoichman

Tuberculosis:

• Medication
• Next Steps

3. Tuberculosis. There is difference in environments
4. Tuberculosis. There is difference in environments. In Nutrition there is report, in CHW there is no report

This is correct behaviour. Tuberculosis ability to change medication, and progress report were developed after the deploy to CHW env took place. The behaviour is correct.

[anvmn]

@LudmilaGoichman

Added my answers. Please see if it all makes sense.

[LudmilaGoichman]

@anvmn Thanks for your answers. It makes sense