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

[16.0] sale_manual_delivery create delivery button missing after first delivery #3254

Open
ChristophAbenthungCibex opened this issue Jul 29, 2024 · 3 comments · May be fixed by #3509
Open
Labels

Comments

@ChristophAbenthungCibex
Copy link
Contributor

ChristophAbenthungCibex commented Jul 29, 2024

Module

sale_manual_delivery

Describe the bug

After confirming the first delivery the button "create delivery" is missing despite that not all parts are delivered.

To Reproduce

Affected versions:
16.0

Steps to reproduce the behavior:

  1. Login as "admin"
  2. Go to "Sales"
  3. Create a new Sale order
  4. Set a customer
  5. Set "Manual delivery" to true
  6. Add a product with quantity 4 (For example: [FURN_6666] Acoustic Bloc Screens)
  7. Add another product with quantity 2 (For example: [CONS_89957] Bolt)
  8. Click "Save" button
  9. Confirm the order with the "Confirm" button
  10. Click "Create delivery" to create a delivery
  11. Set the received quantities to 2 for the first product and to 1 for the second one (only half of products are delivered)
  12. Click "confirm" to confirm the delivery
  13. Click on the smart button "Delivery"
  14. For the created delivery add done amount accordingly (2 for the first product and 1 for the first) and validate the delivery
  15. Go back to the sale order
  16. The button "Create delivery" is missing

Expected behavior
The button "Create delivery" should be there to create a new delivery because only parts are delivered.
The screenshot shows that not all products are delivered (quantities are marked blue) and the button is missing
missing_create_delivery_button

@ChristophAbenthungCibex
Copy link
Contributor Author

Any updates on that issue?

@evemad
Copy link

evemad commented Nov 5, 2024

We have the same problem. Any updates on that issue?

@alexnextev
Copy link

Hi, you can review this pull request #3509 that solves your issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants