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

Inbound Shipment: app crashes when changing status from 'Delivered' to 'Verified' #4941

Open regotaina opened 4 hours ago

regotaina commented 4 hours ago

What went wrong? 😲

After receiving an Inbound Shipment, I made no changes and set status to 'Delivered'. Then changed the quantity received, from 30 to 29, considering for example that 1 vial was damaged during shipping.

App crashes straight away when changed status to from 'Delivered' to Verified'.

Expected behaviour 🤔

App does not crash and stock is updated.

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Go to 'Inbound Shipment'
  2. Choose a shipment that status = 'Shipped'
  3. Change status to 'Delivered'
  4. Adjust the quantity of your first item line and reduce 1 unit.
  5. Change status to 'Verified'.
  6. See error.

Your environment 🌱

regotaina commented 3 hours ago

I was able to reproduce the error twice, repeating the steps above.

I will explore more scenarios, but it's worth investigating this one.

AnushaUp commented 3 hours ago

Can be recreated in all the scenarios🥲 and same for tablet.