-
Notifications
You must be signed in to change notification settings - Fork 247
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
Msi 2035 storepickup #2036
Msi 2035 storepickup #2036
Conversation
@@ -0,0 +1,25 @@ | |||
# InventoryInStorePickupApi module | |||
|
|||
The `InventoryInStorePickupApi` module provides Inventory Management service contracts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
provides service contracts for In-Store Pickup functionality
## Extensibility | ||
|
||
The `InventoryInStorePickupApi` module contains extension points and APIs that 3rd-party developers | ||
can use to provide custom inventory source selection algorithm functionality. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to provide custom inventory source selection algorithm functionality.
instead of " custom inventory source selection algorithm functionality."
it should be something like - customization of In Store Pickup functionality
@@ -0,0 +1,24 @@ | |||
# InventoryInStorePickupAdminUi module | |||
|
|||
The `InventoryInStorePickupAdminUi` module provides Inventory Management service contracts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be rephrased to something like "... extends the Magento Admin UI to add Inventory In Store Pickup functionality."
@@ -0,0 +1,17 @@ | |||
# InventoryInStorePickup module | |||
|
|||
The `InventoryInStorePickup` module provides Inventory Management service contracts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
provides service contracts for In-Store Pickup functionality
"description": "N/A", | ||
"require": { | ||
"php": "~7.1.3||~7.2.0", | ||
"magento/framework": "*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add the dependences.
"magento/module-inventory-in-store-pickup-api": "*", "magento/module-backend": "*", "magento/module-directory": "*", "magento/module-ui": "*"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TadeuRodrigues Test fails due to:
3) Magento\Test\Integrity\DependencyTest::testRedundant
Redundant dependencies found!
Module Magento\InventoryInStorePickup: hard [Magento\InventoryInStorePickupApi]
Module Magento\InventoryInStorePickupAdminUi: hard [Magento\Backend, Magento\Directory, Magento\InventoryInStorePickupApi, Magento\Ui]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm checking.
4b25f54
to
c3db14c
Compare
c3db14c
to
fbab88a
Compare
…ento-engcom/msi into msi-2035-storepickup Conflicts: composer.lock
We are looking to continue development of the Store Pickup feature in a separate branch for now, so I have updated this Pull Request with latest |
Hi @michaelwylde, thank you for your contribution! |
Created base modules for In Store Pickup - 2035
Description
Base modules to be used for In Store Pickup
Fixed Issues (if relevant)