Skip to content

Conversation

@Jeroenll
Copy link
Member

@Jeroenll Jeroenll commented May 12, 2018

Port of #14614

Description

When using GET /V1/orders/items/{id} the parent item isn't set. This PR will add the parent item if the parent_item_id is set.

Fixed Issues (if relevant)

  1. None

Manual testing scenarios

  1. Order a configurable product.
  2. Fetch the ordered simple product using GET /V1/orders/items/{id}.
  3. parent_item_id is set but parent_item isn't.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@Jeroenll Jeroenll added Release Line: 2.3 port-to-2.3 partners-contribution Pull Request is created by Magento Partner Partner: Reach Digital Pull Request is created by partner Reach Digital labels May 12, 2018
@Jeroenll
Copy link
Member Author

@ihor-sviziev Can you also process the port to 2.3-develop?

@magento-engcom-team
Copy link
Contributor

Hi @ihor-sviziev, thank you for the review.
ENGCOM-1532 has been created to process this Pull Request

@Jeroenll Jeroenll changed the title [Port 2.3-develop]Add parent item to order item [Port 2.3-develop] Add parent item to order item May 12, 2018
@magento-engcom-team magento-engcom-team merged commit a6649f8 into magento:2.3-develop May 17, 2018
@magento-engcom-team
Copy link
Contributor

Hi @JeroenVanLeusden. Thank you for your contribution.
Changes from your Pull Request will be available with the upcoming 2.3.0 release.

@Jeroenll Jeroenll deleted the patch-3 branch May 17, 2018 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Partner: H&O Partner: Reach Digital Pull Request is created by partner Reach Digital partners-contribution Pull Request is created by Magento Partner Progress: accept Release Line: 2.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants