-
Notifications
You must be signed in to change notification settings - Fork 930
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
New Serverless Pattern - s3-b2bi-s3- EDI to JSON tranformation #2091
New Serverless Pattern - s3-b2bi-s3- EDI to JSON tranformation #2091
Conversation
s3-b2bi-s3/example-pattern.json
Outdated
"Delete the Amazon S3 input bucket content: <code>aws s3 rm s3://{EDI214InputBucketName} --recursive</code> --region {my-region}", | ||
"Delete the Amazon S3 input bucket content: <code>aws s3 rm s3://{EDI214OutputBucketName} --recursive</code> --region {my-region}", |
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 double check the closing code tags. Also, the second line should refer to the output bucket
s3-b2bi-s3/example-pattern.json
Outdated
"repoURL": "https://github.com/aws-samples/serverless-patterns/tree/main/s3-b2bi-s3", | ||
"templateURL": "serverless-patterns/s3-b2bi-s3", | ||
"projectFolder": "s3-b2bi-s3", | ||
"templateFile": "template.yaml" |
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.
there is no such file in the folder
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.
updated.
s3-b2bi-s3/example-pattern.json
Outdated
"sam deploy -g -t template1.yaml", | ||
"sam deploy -g -t template2.yaml" |
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.
Why do you need two templates? Can they have more descriptive names?
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 have updated the names. Please let me know if they are fine now.
s3-b2bi-s3/README.md
Outdated
- Enter the desired AWS Region. Please refer to the documentation for the list of supported regions. | ||
- Enter `BusinessName`, `Email`, `Phone`, `EDI214InputBucketName`, `EDI214OutputBucketName`, `MyProfileId` and `MyTransformerId`. Please note the last four parameter values should be given as per the output from the `b2bi-stack1` output. | ||
|
||
Once you have run `sam deploy --guided` mode once and saved arguments to a configuration file (samconfig.toml), you can use `sam deploy` in future to use these defaults. |
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 don't understand how this relevant here. Could you elaborate please?
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.
Updated. Please let me know if anything else to change.
Looks good, thanks for your contribution. A DA will merge your request soon |
Grammar issues
Issue #, if available:
Description of changes:
This sample project demonstrates how AWS B2B Data Interchange can transform EDI Files into JSON as per the transformation logic.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.