search

msupply-foundation / open-msupply

Open mSupply represents our most recent advancement in the Logistics Management Information System (LMIS), expanding on more than two decades of development inherited from the well-established legacy of the original mSupply.
https://msupply.foundation/open-msupply/
Other
20 stars 12 forks source link

Cold Chain: Some breaches don't have a Max/Min temperature attached to it #3033

Open regotaina opened 4 months ago

regotaina commented 4 months ago

What went wrong? 😲

We noticed some of the temperature breaches don't have a max/min temperature attached to it on the 'Monitoring' screen:

Screenshot 2024-02-21 at 10 04 23 AM

Maria and Adam noticed there was data on the cold chain app, but for some reason it was not brought across oms.

Expected behaviour 🤔

All breaches should have a min/max temperature associated with.

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Go to 'Cold chain'
  2. Click on 'Monitoring' > breaches
  3. See column 'Max/min' empty for some breaches

Your environment 🌱

regotaina commented 4 months ago

@marthakeezy do you have more screenshots from your set up? Please share if you have any cases like this one

marthakeezy commented 4 months ago

@regotaina I haven't seen this bug before yesterday, but will pop some screenshots in this thread if I notice anything!

Chris-Petty commented 4 months ago

@adamdewey what do you mean tagged? As in including the temperature value?

Vibing normal until concrete it's OMS? Though that assumes investigation has some sorta priority in CCA...

adamdewey commented 4 months ago

@Chris-Petty - eek, my bad, I think that comment is meant for another issue where the breach icons weren't being displayed properly! Will remove..

adamdewey commented 4 months ago

And yah, I would say 'Normal' 👍

roxy-dao commented 4 months ago

Is it still happening?

marthakeezy commented 4 months ago

@roxy-dao yes, I noticed a few on my end earlier today

image

roxy-dao commented 4 months ago

Looks like the temperature logs are missing for these breaches. Not sure if this is because of om or CCA since I don't have the data to check 👀

jmbrunskill commented 2 months ago

Assigning to 2.1 as we don't have repro steps. And not high priority.