Skip to content

Conversation

@thejackwu
Copy link

No description provided.

@calvinhughes
Copy link

According to the most recent docs, the correct tag to use is ResidentialAddress instead of ResidentialAddressIndicator.

Also, I think we should try to follow the behaviour of the UPS API for the passed options. The default behaviour for the ResidentialAddress tag when it's not passed is that it's treated as commercial and this is reclassified by UPS. With this change we're marking every address as residential, so I think we should only add this tag if some option is passed to the address builder (like residential_address)

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.

2 participants