-
Notifications
You must be signed in to change notification settings - Fork 47
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
Extended the description in the incoming shipment creation flow #919
Open
Chinmay2107
wants to merge
2
commits into
hotwax:implementations-pub
Choose a base branch
from
Chinmay2107:implementations-pub
base: implementations-pub
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,20 +8,19 @@ This document highlights how Transfer Orders (TOs) will be managed after a recom | |
1. **Creating and Accepting Transfer Orders** | ||
- After Transfer Order recommendations are generated, the merchandising team reviews and accepts the needed TOs. | ||
- | ||
- The merchandising team will create CSV file and upload them into HotWax using EXIM. | ||
- Once the file is uplaoded an existing job within HotWax OMS creates the corresponding TOs. | ||
- **Note**: More detail is required on this process. | ||
- The merchandising team will create CSV file and upload them into HotWax using [Import Transfer Order]([url](https://mephisto-uat.hotwax.io/commerce/control/ImportData?configId=IMP_TRANSFER_ORD)) MDM. | ||
- Once the file is uploaded an existing Import Transfer Order job within HotWax OMS creates the corresponding TOs. | ||
- An existing job invokes `bulkApproveTransferOrders` service that approves the TOs in created status and reserves the inventory for the products. | ||
|
||
2. **Fulfilling Transfer Orders** | ||
- The store fulfills these accepted Transfer Orders and creates shipments for them with `"shipmentTypeId": "OUT_TRANSFER"`. | ||
- This process already exists in HotWax OMS. | ||
- **Note**: More detail is required on this process. | ||
|
||
3. **Scheduled Job for Outgoing Transfers** | ||
- A scheduled job will be created to pull all new `"OUT_TRANSFER"` shipments in the `"PURCH_SHIP_SHIPPED"` status. | ||
- These shipments will be placed at an SFTP location for further processing. | ||
- **Note**: The format of this file needs to be identified. Ideally, it should match the format used by the existing jobs. This is JSON | ||
- **Note**: This job may be helpful here `generate_TransferOrderFulfilledItemsFeed"` | ||
- **Note**: The`generate_TransferOrderFulfilledItemsFeed"` service generates the feed in JSON format. | ||
|
||
4. **Using NiFi to Create Incoming Transfers** | ||
- A NiFi flow will be created to process the files placed at the SFTP location. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The NiFi has been created and tested. Details of the flow should be added here. |
||
|
@@ -31,7 +30,7 @@ This document highlights how Transfer Orders (TOs) will be managed after a recom | |
- The file created by NiFi will be uploaded into OMS using configID: `IMP_SHIPMENT` and job: `createIncomingShipment` | ||
|
||
### JOLT Transformation for Incoming Transfers | ||
The following JOLT transformation is used create the new `"IN_TRANSFER"` shipment: | ||
The following JOLT transformation is used to create the new `"IN_TRANSFER"` shipment: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is a part of the NiFi flow. |
||
|
||
```json | ||
[ | ||
|
@@ -56,4 +55,4 @@ The following JOLT transformation is used create the new `"IN_TRANSFER"` shipmen | |
|
||
**Note**: The `OUT_TRANSFER` and the `IN_TRANSFER` are linked to the same order and shipment package, therefore most of the data will remain the same. | ||
|
||
This JOLT transformation is designed based on the requirements for [creating incoming shipments](https://docs.google.com/document/d/1hOK8CAYw2TpBYudrHVOrFc5EbcPQAns2xB-VaRLDlZU/edit?usp=sharing) (`createincomingshipment`). | ||
This JOLT transformation is designed based on the requirements for [creating incoming shipments](https://docs.google.com/document/d/1hOK8CAYw2TpBYudrHVOrFc5EbcPQAns2xB-VaRLDlZU/edit?usp=sharing) (`createincomingshipment`). |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 think line can also be deleted.