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

Order ID displayed in 'My orders' identical to the order ID in Magento #2743

Closed
2 of 6 tasks
GabiDivante opened this issue Apr 16, 2019 · 15 comments
Closed
2 of 6 tasks
Labels
1: Easy feature request Requests for new features. Please be as specific as possible and provide proposal API if it you can QA approved after merge Testers will add this label after positive check on merged changes vs-hackathon Tasks for the Hackathon
Milestone

Comments

@GabiDivante
Copy link
Collaborator

What is the motivation for adding / enhancing this feature?

Currently, order ID displayed in 'My orders' is different from order ID in Magento. It is confusing for the customer when he sees one order ID in 'My orders' and gets a different one in the email order confirmation from Magento.

What are the acceptance criteria

  • Order ID displayed in My orders is the same as in Magento Orders list.

Can you complete this feature request by yourself?

  • YES
  • NO

Which Release Cycle state this refers to? Info for developer.

Pick one option.

  • This is a normal feature request. This should be available on https://test.storefrontcloud.io and then after tests this can be added to next Vue Storefront version. In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
  • (Pick this option only if you're sure) This is an important improvement request for current Release Candidate version on https://next.storefrontcloud.io and should be placed in next RC version. In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
  • (Pick this option only if you're sure) This is a critical improvement request for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version. In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Additional information

@GabiDivante GabiDivante added the feature request Requests for new features. Please be as specific as possible and provide proposal API if it you can label Apr 16, 2019
@pkarw pkarw added vs-hackathon Tasks for the Hackathon 1: Easy labels Apr 17, 2019
@pkarw pkarw closed this as completed Apr 25, 2019
pkarw added a commit that referenced this issue Apr 27, 2019
@patzick patzick added this to the 1.10.0-rc.1 milestone Apr 28, 2019
pkarw added a commit that referenced this issue May 3, 2019
@GabiDivante
Copy link
Collaborator Author

@mdesmet @pkarw we need to add the correct Order ID on the order summary page as well
Selection_027

@GabiDivante GabiDivante reopened this May 10, 2019
@mdesmet
Copy link
Contributor

mdesmet commented May 10, 2019

HI Gabi, I mentioned that open issue on the fix indeed. I'll have a look at it tomorrrow, must be not too hard to fix.

@GabiDivante
Copy link
Collaborator Author

@mdesmet Great! Thank you :)

@pkarw
Copy link
Collaborator

pkarw commented May 10, 2019

@mdesmet please just change the orderData.entity_id to orderData.increment_id and You’re there

@mdesmet
Copy link
Contributor

mdesmet commented May 10, 2019

@pkarw : from what I remember it was more complicated, the magento api returns only the internal id (see o2m.js).

@pkarw
Copy link
Collaborator

pkarw commented May 10, 2019

You might be right...

@mdesmet
Copy link
Contributor

mdesmet commented May 11, 2019

I don't really have time now to solve it right now, working on another project but here is my analysis:

We would have to extend the magento2-rest-client with support for retrieving the order based on the increment id returned by the order creation endpoint. See this Stack exchange with the solution: https://magento.stackexchange.com/questions/210000/retrieve-entity-id-from-rest-api-using-order-id

Then we can call this api in the o2m.js and change the bakendOrderId to the correct order id.

@Flyingmana
Copy link
Contributor

shouldnt this be more of a cosmetic then a functional change?
vue-storefront is intended to connect with different systems, and its common that an Order has an entity ID, but exposes a different ID to the customer, which often allows more flexibility.

So it may make sense, that vue-storefront adds a new property to differentiate between entity_id and shown order_id

@mdesmet
Copy link
Contributor

mdesmet commented May 24, 2019

I wish it was just a cosmetic change but the response from the order endpoint only returns the internal increment id, no other properties, so we have to get the external order id somewhere else. IMHO the analysis still stands.

@patzick patzick added the QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. label May 25, 2019
@patzick
Copy link
Collaborator

patzick commented May 25, 2019

Thanks @Flyingmana !

@patzick patzick closed this as completed May 25, 2019
@GabiDivante
Copy link
Collaborator Author

GabiDivante commented May 27, 2019

@Flyingmana I'm testing this solution on the branch develop and still order id on the thankyou page is different than in my orders, can you check it?

@GabiDivante GabiDivante reopened this May 27, 2019
@GabiDivante GabiDivante removed the QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. label May 27, 2019
@patzick
Copy link
Collaborator

patzick commented May 27, 2019

i've spoken with Gabi, i merged that to separate branch to faster merge that huge abount of PRs, no worries here :)

@patzick patzick closed this as completed May 27, 2019
@patzick patzick added the QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. label May 27, 2019
@mdesmet
Copy link
Contributor

mdesmet commented May 27, 2019

@patzick: I meant to comment on this one, actually this change doesn't work as orderData.increment_id is not defined at that point. We have to fetch it from magento api. @Flyingmana is on it already.

@Flyingmana
Copy link
Contributor

added this fetch via vuestorefront/vue-storefront-api#258

@GabiDivante GabiDivante added QA approved after merge Testers will add this label after positive check on merged changes QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. and removed QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. QA approved after merge Testers will add this label after positive check on merged changes labels Jun 5, 2019
@alinadivante alinadivante added QA approved after merge Testers will add this label after positive check on merged changes and removed QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. labels Jun 19, 2019
@alinadivante
Copy link
Collaborator

@Flyingmana could you look at this feature request #3082 ? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1: Easy feature request Requests for new features. Please be as specific as possible and provide proposal API if it you can QA approved after merge Testers will add this label after positive check on merged changes vs-hackathon Tasks for the Hackathon
Projects
None yet
Development

No branches or pull requests

6 participants