Skip to content

Added: Suite scripts to export store to store transfer orders Json file #197

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

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

Shinde-nutan
Copy link

@Shinde-nutan Shinde-nutan commented Apr 10, 2025

Related issue: #194

@gurveenbagga gurveenbagga changed the title Added the two suite script for store to store TO Added Suite scripts for store to store TO CSV and JSON Apr 10, 2025
@Shinde-nutan Shinde-nutan changed the title Added Suite scripts for store to store TO CSV and JSON Added: Suite scripts to export store to store transfer orders Json file #198 Apr 23, 2025
@Shinde-nutan Shinde-nutan changed the title Added: Suite scripts to export store to store transfer orders Json file #198 Added: Suite scripts to export store to store transfer orders Json file Apr 23, 2025
@Shinde-nutan Shinde-nutan marked this pull request as ready for review April 23, 2025 10:20
@gurveenbagga gurveenbagga requested a review from rehan3586 May 6, 2025 07:28

var connection = sftp.createConnection({
username: sftpUserName,
keyId: sftpKeyId,
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Shinde-nutan, We are not using keyId while setting up the connection with SFTP. Please use secret instead of keyId

};

mapContext.write({
key: internalid,
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Shinde-nutan, we should append the line ID with the internal ID to make the key ID unique.

Copy link
Author

Choose a reason for hiding this comment

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

When creating a Transfer Order in JSON format, we use "internalId" as the key. This allows the subsequent method (shuffle) to group all maps with the same key. This grouping will assist us in the reduce method, enabling the creation of a nested JSON structure for the transfer order.

Additionally, as we discussed, I checked and confirmed that while preparing the JSON files, we are not encountering any discrepancies with the items. However, when we attempt to create a CSV file, the execution control does not enter the reduce state, which results in the transfer order not being exported.

@gurveenbagga gurveenbagga linked an issue May 7, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Export Store to Store Transfer Order from NetSuite to Hotwax
3 participants