INT - B-22138 - Duty Location Results Should Not Include ZIP Codes for PO Boxes #14482
+2,898
−9
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
B-22138
Summary
This is to add
is_po_box
column tore_us_post_regions
table to keep record of postal codes being a PO Box or not. Query for searching duty locations in duty location dropdown(customer & office share the same query) have been updated to filter out duty locations whereis_po_box
is true.is_po_box
column tore_us_post_regions
table and updated existing rows that need to haveis_po_box
as true (default is false).is_po_box
is_po_box
tov_locations
view.is_po_box = true
.How to test
92137
when searching for duty locations and verify that exact match does not exist. This zip is a PO Box so it should not be available in the results.San Diego
and verify a result with92137
does not show.Screenshots