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
23 stars 15 forks source link

GAPS: Link stock data to specific storage temperature ranges #4367

Closed adamdewey closed 2 weeks ago

adamdewey commented 4 months ago

Is your feature request related to a problem? Please describe 👀

Part of Epic https://github.com/msupply-foundation/open-msupply/issues/4362

Describe the solution you'd like 🎁

You can already assign multiple locations to a cold chain asset:

Image

So just need to add temperature_range to a location. This would need to be stored in OMS central, on the location table (much like the name_oms_fields extension of the name table)

Show the temperature range in brackets after the location name, when selecting the location for a stock line (i.e. in inbound shipment)

This can then be used in dashboard queries

Additional context 💌

User story A.6.2
As a… Health Care Worker (HCW) or warehouse worker
I want… Stock inventory data to be associated with the correct cold storage location
So that… I can see exactly what stock each CCE is storing at any given time

Solution A.6.2

The system will allow for users to make the association between stock storage locations and CCE by providing a mechanism to create new stock storage locations that represent the cold storage within CCE.

Multiple locations will be possible for each CCE and the user will be able to specify which type of cold storage they are associated with - e.g. +5 °C and -20 °C for hybrid refrigerator / freezers.

Note: the exactly what stock each CCE is storing would be nice but not required at this stage 🙏

Moneyworks Jobcode 🧰

CCEIDEV

lache-melvin commented 3 weeks ago

Some decision changes since above!

Somewhere we will create a mapping to be used for the dashboards, deriving the known temperature ranges from the location_types to line up with the rest of the GAPS data

mark-prins commented 3 weeks ago

Thanks Laché that's great! There is some practical cross-over with the storage capacity properties of cold chain equipment:

image

and

image

We've talked about implementing a dashboard or report to show available volume and volume used. The question is whether we bring across the location volume from OG, or use the volume property from the asset. While not ideal, we can make do with the latter for now.

jmbrunskill commented 3 weeks ago

We've talked about implementing a dashboard or report to show available volume and volume used. The question is whether we bring across the location volume from OG, or use the volume property from the asset.

I think there's potential for locations to be broken down more than what's in the asset. E.g. you could a location for each shelf in a fridge. Ideally the system should warn or stop the user from a assigning more volume that the asset actually has. I agree we can live with using asset volumes for now, but thought I'd mention...

adamdewey commented 3 weeks ago

Ideally the system should warn or stop the user from a assigning more volume that the asset actually has.

Agree, also ideally the system would actually create a stock location with the appropriate temperature range when creating an asset..

CarlosNZ commented 3 weeks ago

5179

CarlosNZ commented 2 weeks ago

Just re-opening until I add in the front-end mutation to update the cold storage setting on location.

Will also need coldStorageType GraphQL node to get the list of storage types