Skip to content
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

PBC-3110 Added description of data mapping between quote/order objects on Spryker side and Vertex API. #2252

Merged
merged 13 commits into from
Oct 30, 2023

Conversation

bohdan-turchyk-spryker
Copy link
Contributor

@bohdan-turchyk-spryker bohdan-turchyk-spryker commented Oct 16, 2023

PR Description

Can be merged when https://spryker.atlassian.net/browse/PBC-3118 and https://spryker.atlassian.net/browse/PBC-3121 are released

Ticket:

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

@bohdan-turchyk-spryker bohdan-turchyk-spryker added TW review needed PR needs Technical writer's review updated PR contains meaningful changes to some document labels Oct 16, 2023
@lenadoc lenadoc added in TW review PR is in review by a technical writer do not merge PR is not ready for merge and removed TW review needed PR needs Technical writer's review labels Oct 19, 2023
### Quote/Order object to Vertex API mapping

Mapping of Spryker Quote/Order transfer object to Vertex API request format is described in the following table:

Copy link
Contributor

Choose a reason for hiding this comment

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

@bohdan-turchyk-spryker can you please provide a few words with the context of why we have this mapping, e.g. when to use it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Please follow this data mapping format defined here if you need to implement any custom expander plugins for Vertex integration.

I can add this line in the doc if needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added an explanation, please check.

@lenadoc lenadoc removed do not merge PR is not ready for merge in TW review PR is in review by a technical writer labels Oct 27, 2023
@lenadoc lenadoc merged commit 918b721 into master Oct 30, 2023
13 checks passed
@lenadoc lenadoc deleted the feature/pbc-3110/quote-order-mapping-to-vertex-api branch October 30, 2023 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
updated PR contains meaningful changes to some document
Development

Successfully merging this pull request may close these issues.

3 participants