Skip to content

MAGETWO-11669: API salesRefundInvoiceV1 does no save invoice ID on credit memo #11687

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

Closed
wants to merge 1 commit into from
Closed

Conversation

nmalevanec
Copy link
Contributor

Description

Fix for API salesRefundInvoiceV1 does no save invoice ID on credit memo.

Fixed Issues (if relevant)

  1. API salesRefundInvoiceV1 does no save invoice ID on credit memo #11669: API salesRefundInvoiceV1 does no save invoice ID on credit memo

Manual testing scenarios

  1. Place an order with an online payment method (PayPal).
  2. Create an online invoice for the order.
  3. Use API method salesRefundInvoiceV1 to create a refund for the invoice.
  4. Check newly added row in sales_creditmemo table in Your Magento database. Field 'invoice_id' should content id of invoice credit memo was created for.

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)

@ishakhsuvarov ishakhsuvarov self-assigned this Oct 24, 2017
@ishakhsuvarov ishakhsuvarov added this to the October 2017 milestone Oct 24, 2017
@ishakhsuvarov ishakhsuvarov added Release Line: 2.2 2.2.x Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release labels Oct 24, 2017
@ishakhsuvarov
Copy link
Contributor

Duplicates #11670

@nmalevanec
Copy link
Contributor Author

Close this pull-request as duplicate #11670

@nmalevanec nmalevanec closed this Oct 27, 2017
@nmalevanec nmalevanec deleted the MAGETWO-11669 branch October 27, 2017 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Progress: needs update Release Line: 2.2 Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants