Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

fix(orders): parse ShippingAddress as string #367

Merged
merged 2 commits into from
May 4, 2021
Merged

fix(orders): parse ShippingAddress as string #367

merged 2 commits into from
May 4, 2021

Conversation

Roman991
Copy link
Contributor

@Roman991 Roman991 commented May 4, 2021

part2 or #365

All properties in ShippingAddress are filled by customers and should be strings

https://docs.developer.amazonservices.com/en_US/orders-2013-09-01/Orders_Datatypes.html#Address

@moltar
Copy link
Contributor

moltar commented May 4, 2021

Great, thanks.

Can you please also add some test cases for the data itself?

Please see my previous PR where I modify the XML file and added the phone. You can just edit one of the entries too, and then replace all of the values with number-looking values.

Then run npm t -- path/to/test/file.ts -u to update the snapshots.

Thanks.

<ShippingAddress>
<Name>Jane Smith</Name>
<AddressLine1>22</AddressLine1>
<AddressLine2>kebab st</AddressLine2>
Copy link
Contributor

Choose a reason for hiding this comment

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

Delicious.

@moltar moltar merged commit 7909ef4 into ScaleLeap:master May 4, 2021
github-actions bot pushed a commit that referenced this pull request May 4, 2021
## [1.9.49](v1.9.48...v1.9.49) (2021-05-04)

### Bug Fixes

* **orders:** parse `ShippingAddress` as string ([#367](#367)) ([7909ef4](7909ef4))
@github-actions
Copy link

github-actions bot commented May 4, 2021

🎉 This PR is included in version 1.9.49 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants