-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Picked OS: Can no longer edit stock in invoice line if all has been issued #3704
Comments
Sorry the full reasoning is a bit lost to me now, but at some point there was consensus on it, perhaps we need some documentation about it but we would need all stakeholder to participate, including mSupply pps. I feel like this area is being discussed all the time and perhaps needs a diagram and explanation ?
Stock line still exists. We should display that stock line in the list. I think it's deliberately being filtered out. |
Btw do we have front end test for this ? |
I think should definitely be documented since it is being discussed a lot. |
no - the f/e tests are usually around logic and component functionality. this would be a b/e test or f/e-b/e integration test, if we had such a thing. |
|
Tested on RC4 🧪 How has/should this change been tested?
|
What went wrong? 😲
Stock lines disappearing from invoice line after allocating all the stock since stock leaves inventory at
Picked
, but we are still allowing the user to edit the invoice, so the user should still be able to edit the stock.Q. Why does OMS
picked
functionality have to match OGcn
functionality? Why can stock not still be in inventory but just unavailable?Expected behaviour 🤔
The user should still be able to edit the stock line!
How to Reproduce 🔨
Steps to reproduce the behaviour:
Outbound Shipment
Picked
Your environment 🌱
The text was updated successfully, but these errors were encountered: