Skip to content
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

Closed
roxy-dao opened this issue Apr 29, 2024 · 6 comments
Closed
Assignees
Labels
bug Something is borken needs architecture/solution Needs wider dev input on general solution Team Ruru 🦉 Andrei, Roxy, Ferg Tester: Taina
Milestone

Comments

@roxy-dao
Copy link
Contributor

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 OG cn functionality? Why can stock not still be in inventory but just unavailable?

Screenshot 2024-04-30 at 10 08 37

Expected behaviour 🤔

The user should still be able to edit the stock line!

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Create an Outbound Shipment
  2. Add invoice line
  3. Issue all stock from stock line
  4. Change status to Picked
  5. Go back to invoice line
  6. See that stock line has disappeared

Your environment 🌱

  • Open mSupply Version:
  • Legacy mSupply Central Server Version:
  • Platform:
  • Database type:
@roxy-dao roxy-dao added bug Something is borken needs triage labels Apr 29, 2024
@roxy-dao roxy-dao changed the title A brief clear title describing the bug from a user's perspective Picked OS: Can no longer edit stock in invoice line if all has been issued Apr 29, 2024
@roxy-dao roxy-dao added needs architecture/solution Needs wider dev input on general solution needs design Needs PO/UI/UX designer input labels Apr 29, 2024
@andreievg
Copy link
Collaborator

Q. Why does OMS picked functionality have to match OG cn functionality?

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 ?

Why can stock not still be in inventory but just unavailable?

Stock line still exists. We should display that stock line in the list. I think it's deliberately being filtered out.

@andreievg
Copy link
Collaborator

Btw do we have front end test for this ?

@roxy-dao
Copy link
Contributor Author

roxy-dao commented Apr 29, 2024

I think should definitely be documented since it is being discussed a lot.

@mark-prins
Copy link
Collaborator

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.

@andreievg
Copy link
Collaborator

I think should definitely be documented since it is being discussed a lot.

msupply-foundation/msupply_docs#32

@DhanyaHerath DhanyaHerath added this to the V2.0.0-rc2 milestone Apr 30, 2024
@roxy-dao roxy-dao added Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release and removed needs triage labels Apr 30, 2024
@DhanyaHerath DhanyaHerath removed the Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release label Apr 30, 2024
@lache-melvin lache-melvin added the Team Ruru 🦉 Andrei, Roxy, Ferg label May 1, 2024
@andreievg andreievg self-assigned this May 2, 2024
@roxy-dao roxy-dao closed this as completed May 5, 2024
@roxy-dao roxy-dao modified the milestones: V2.0.0-rc3, V2.0.0 May 5, 2024
@regotaina
Copy link
Contributor

Tested on RC4
PASSED THE TEST 💯

🧪 How has/should this change been tested?

  • Create an Outbound Shipment
  • Add invoice line
  • Issue all stock from stock line
  • Change status to Picked
  • Go back to invoice line
  • See that stock line has disappeared

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is borken needs architecture/solution Needs wider dev input on general solution Team Ruru 🦉 Andrei, Roxy, Ferg Tester: Taina
Projects
None yet
Development

No branches or pull requests

6 participants