Skip to content

added residential address indicator at address builder #10

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.

Sign up for GitHub

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
thejackwu wants to merge 5 commits into
base: master
Choose a base branch
from
Open

added residential address indicator at address builder #10

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
bfe5b3c
added residential address indicator at address builder
thejackwu May 13, 2019
1c56cbd
changed to empty tag instead
thejackwu May 14, 2019
0462645
remove unused comment
thejackwu May 14, 2019
ddd4a5b
added warning message on unoffical API usages
thejackwu May 14, 2019
9beb84b
remove warning since it is included in UPS API doc
thejackwu May 15, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions lib/ups/builders/address_builder.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,10 @@ def email_address
# @return [Ox::Element] XML representation of the current object
def to_xml
Element.new('Address').tap do |address|
# By setting the residential address indicator tag.
# We can trigger UPS to check for any surcharge related to that area
# and apply the charge automically.
address << element_with_value('ResidentialAddressIndicator', '')
address << address_line_1
address << address_line_2
address << email_address if opts[:email_address]
Expand Down