Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

ronak2ram
Copy link
Member

@ronak2ram ronak2ram commented Jan 26, 2019

This PR is a:

  • New topic
  • Content update
  • Content fix or rewrite
  • Bug fix or improvement

Summary

When this pull request is merged, it will correct delete Inventory Low Quantity Notification Api

Additional information

Correct delete Inventory Low Quantity Notification Api

List all affected URLs

whatsnew
Corrected the route of the POST /V1/inventory/low-quantity-notifications-delete endpoint.

@magento-cicd2
Copy link
Contributor

An admin must run tests on this PR before it can be merged.

@osrecio osrecio self-requested a review January 28, 2019 11:14
@osrecio osrecio self-assigned this Jan 28, 2019
Copy link
Member

@osrecio osrecio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ronak2ram thanks for your contribution.

I checked with MSI REPO and your modification is ok.

Thanks this PR will be processed soon.

@keharper keharper self-assigned this Jan 28, 2019
@keharper keharper added 2.3.x Magento 2.3 related changes Technical Updates to the code or processes that alter the technical content of the doc labels Jan 28, 2019
@keharper
Copy link
Contributor

running tests

@keharper keharper merged commit 59ac260 into magento:master Jan 28, 2019
@ghost
Copy link

ghost commented Jan 28, 2019

Hi @ronak2ram, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@keharper
Copy link
Contributor

Great catch, @ronak2ram . That was a late change in the development cycle.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.3.x Magento 2.3 related changes Partner: Krish TechnoLabs partners-contribution PR created by Magento partner Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants