Comments (21)
Confirming the issue is present in 2.4-develop, screenshots below from the instance above:
from magento2.
@engcom-Bravo it's not consistent with the rest of Magento behavior then.
Let's take the case below:
- Create an order containing: 1 physical item of quantity = 2 (NO VIRTUAL ITEM)
- Invoice the order
- Refund 1 quantity of the physical item
- Ship 1 quantity of the physical item
In that case, the order becomes Complete:
On top of that if you create another order like this:
- Create an order containing: 1 virtual item of quantity = 5 (NO PHYSICAL ITEM)
- Invoice the order
The order is complete as well.
Knowing that, combination of both should be complete too.
from magento2.
Hi @digitalpianism. Thank you for your report.
To speed up processing of this issue, make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:
@magento give me 2.4-develop instance
- upcoming 2.4.x release- For more details, review the Magento Contributor Assistant documentation.
- Add a comment to assign the issue:
@magento I am working on this
- To learn more about issue processing workflow, refer to the Code Contributions.
Join Magento Community Engineering Slack and ask your questions in #github channel.
🕙 You can find the schedule on the Magento Community Calendar page.
📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket.
from magento2.
@magento give me 2.4-develop instance
from magento2.
Hi @digitalpianism. Thank you for your request. I'm working on Magento instance for you.
from magento2.
Hi @digitalpianism, here is your Magento Instance: https://6fc1486a44c08bdcd128df01c88544ca.instances-prod.magento-community.engineering
Admin access: https://6fc1486a44c08bdcd128df01c88544ca.instances-prod.magento-community.engineering/admin_b24a
Login: 15429f36
Password: 18b19d16f0d9
from magento2.
Hi @engcom-Bravo. Thank you for working on this issue.
In order to make sure that issue has enough information and ready for development, please read and check the following instruction: 👇
- 1. Verify that issue has all the required information. (Preconditions, Steps to reproduce, Expected result, Actual result).
- 2. Verify that issue has a meaningful description and provides enough information to reproduce the issue.
- 3. Add
Area: XXXXX
label to the ticket, indicating the functional areas it may be related to. - 4. Verify that the issue is reproducible on
2.4-develop
branchDetails
- Add the comment@magento give me 2.4-develop instance
to deploy test instance on Magento infrastructure.
- If the issue is reproducible on2.4-develop
branch, please, add the labelReproduced on 2.4.x
.
- If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and stop verification process here! - 5. Add label
Issue: Confirmed
once verification is complete. - 6. Make sure that automatic system confirms that report has been added to the backlog.
from magento2.
@magento give me 2.4-develop instance
from magento2.
Hi @engcom-Bravo. Thank you for your request. I'm working on Magento instance for you.
from magento2.
Hi @engcom-Bravo, here is your Magento Instance: https://6fc1486a44c08bdcd128df01c88544ca.instances-prod.magento-community.engineering
Admin access: https://6fc1486a44c08bdcd128df01c88544ca.instances-prod.magento-community.engineering/admin_2102
Login: dea1c8b9
Password: 65ecb079ce48
from magento2.
Hi @digitalpianism,
Thank you for reporting and collaboration.
Verified the issue on Magento 2.4-develop instance and the issue is not reproducible.Kindly refer the screenshots.
Steps to reproduce
- Create an order containing: 1 physical item of quantity = 2 and 1 virtual item of quantity = 5
- Invoice the order
- Refund 1 quantity of the physical item
- Ship 1 quantity of the physical item
The order is still in Processing because we have one more quantity for the physical product to ship.
Could you please let us know if we are missing anything.
Thanks.
from magento2.
@magento give me 2.4-develop instance
from magento2.
Hi @engcom-Bravo. Thank you for your request. I'm working on Magento instance for you.
from magento2.
Hi @engcom-Bravo, here is your Magento Instance: https://6fc1486a44c08bdcd128df01c88544ca.instances-prod.magento-community.engineering
Admin access: https://6fc1486a44c08bdcd128df01c88544ca.instances-prod.magento-community.engineering/admin_a8e6
Login: 96ce0477
Password: a63125bbdf3c
from magento2.
Hi @digitalpianism,
Thanks for your update.
Verified the issue on Magento 2.4-develop instance and the issue is reproducible.
The Order is still in Processing state when we have combination of both physical products and virtual products.
Hence Confirming the issue.
Thanks.
from magento2.
✅ Jira issue https://jira.corp.adobe.com/browse/AC-11691 is successfully created for this GitHub issue.
from magento2.
✅ Confirmed by @engcom-Bravo. Thank you for verifying the issue.
Issue Available: @engcom-Bravo, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself.
from magento2.
❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.adobe.com/browse/AC-11691
from magento2.
@magento I am working on this
from magento2.
@magento run all tests
from magento2.
I have started to have some feedbacks for this issue and I will put a report on where we are at this point.
- it is possible to refund an item before it has been shipped
- it is also possible to refund an item after it has been shipped
These 2 operations can be performed from the backend using the credit memo backend screen.
As a consequence, when a product has been refunded, it can overlap with the shipment data (in which case, using the tests to reproduce the issue, the order is expected to remain to processing)
it can be separate from the shipment data, we refund an item that was not sent yet (the credit memo is expected to adjust the shipping fees to 0 in this case) and the order status would be expected to be set to complete
--> the conclusion of this finding is that unless we can flag is a refund is for a shipped item or not, then we can't change the order status to complete.
--> the focus with this issue was to verify virtual items did not prevent the order to be set to complete if all the physical items in the order were handled by shipment or refund. But the fact we are not able to separate refund and shipment data leads us to be still at the original place.
from magento2.
Related Issues (20)
- Issue : Incorrect Data on Magento 2 Dashboard HOT 9
- Multiple HTML attributes not editable on Categories HOT 6
- Legacy HTML content is imported to Page Builder as HTML code, not a Text element HOT 10
- "Area code not set" after update to 2.4.5-p8 HOT 14
- Payment Failed Emails cannot be styled with css by including the default email header HOT 9
- Add "algorithm" support to db_schema.xml HOT 1
- Upgrade to Magento 2.4.7 error in module-payment-services-base HOT 10
- CSP report_only field is missing in backend configuration HOT 5
- Navigating in the categories tree leads to errors in Redis: "Redis session exceeded concurrent connections" HOT 4
- [Issue] Update jquery.metadata.js DOM text reinterpreted as HTML HOT 2
- Composer showing error during version upgrade from 2.3.6-p1 to 2.4.7-p1 HOT 5
- Admin captcha does not load due to CSP when admin and store base_url are different. HOT 2
- Watermark turning images cyan HOT 5
- Magento 2 dashboard issue HOT 4
- GraphQL categories field under ProductInterface contains disabled categories HOT 2
- 2.4.7-p1 incompatible with magento/extension-b2b HOT 3
- Fedex integration doesn't support Track API being a separate Project in FedEx account HOT 2
- Dashboard Order graph animates into final size HOT 12
- Uncaught exception 'Symfony\Component\Intl\Exception\ResourceBundleNotFoundException' with message HOT 8
- Category fields missing data when in deeply (>1) nested GraphQL fragments HOT 7
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from magento2.