Fix enrolment summary rule for pregnancy program

avniproject / JSSCP

0 stars 0 forks source link

Fix enrolment summary rule for pregnancy program #344

Open Gojo-Taqi opened 3 months ago

Gojo-Taqi commented 3 months ago

Rule uuid - '20016e73-3115-439d-8d8b-3bd4ab5c24b1'

[ ] Have backup bundles of both the orgs(uat and prod)
[ ] Upload the bundle to uat
[ ] The stack trace from production says "Cannot read property 'getValue' of undefined", which means in the summary rule, wherever it is calling the function 'getValue()', some values are coming as undefined.
[ ] Identify the faulty line of code and fix it.
[ ] Have the before-fix code and the after-fix code in the card as comments.

Gojo-Taqi commented 3 months ago

Even though all the concepts that are using getValue() are mandatory fields, they some how manage to come as undefined, and later when 'getValue' is called, it gives the above error.

BEULAHEVANJALIN commented 3 months ago

It's failing when trying to get the values of LMP and EDD.

Found out that 10 people didn't have this LMP and EDD info:

0f0d839f-4bea-413c-b9f6-d8e8314ce6f3, Ishneha, Dhurve
35267348-8f37-4253-900a-b9c6417a97a0, Umita, Baiga
4880cb36-1f2b-4895-850f-b101fdcb37c8, Devi, Dhruv
51fb1b55-b07f-44bb-8d7b-47c0020da364, Laxmi, Gond
8e7a8063-8ac9-48c0-9404-e3b064f44602, Punam, Porte
985d2f62-e7cd-4907-a9fb-05c3b077e48d, Kaushilya, Sandram
b0038184-762e-4bb1-97d9-f1eb0beb97e6, Manmati, Baiga
b17ad5c4-ad95-4f87-9976-3c43ca644c24, Samari, Dhruve
d16b3ffc-34c0-486b-a8ea-361ace0fa534, Puniya Bai, Urav
e82b96f6-5825-4410-943c-929be95fa8b2, Soniya, Kol

BEULAHEVANJALIN commented 3 months ago

select *
from rule_failure_telemetry
where rule_uuid = '20016e73-3115-439d-8d8b-3bd4ab5c24b1'
  and is_closed = false;

update rule_failure_telemetry
set is_closed        = true,
    closed_date_time = now()
where rule_uuid = '20016e73-3115-439d-8d8b-3bd4ab5c24b1';

BEULAHEVANJALIN commented 2 months ago

Uploaded the UAT bundle to prod on April 8th

Dinesh2019 commented 2 months ago

Seems that this issue popping up again, despite having null checks :( We need to ask them to upload their mobile databases for further investigation.