Skip to content

Check UPS mail drop boxes #4866

Closed
Closed

Description

@bhousel

I'm feeling mostly good about how mail boxes look in NSI v5, except for UPS.

UPS looks weird because we use a template to expand all of the post offices into mail drop locations. The post office is branded/named/operated by "The UPS Store" (fine) but the mail drop boxes really should just be branded/named/operated by "UPS" (which has a different QID from the UPS Store).

I was hoping to avoid this, but it looks like I need to figure out a way to make the templates slightly smarter. We need a way of saying which items to not expand from the templateSource. 😓

"operators/amenity/post_box": [
{
"templateSource": "operators/amenity/post_office",
"templateTags": {
"amenity": "post_box",
"name": ""
}
}
]

Screen Shot 2021-01-18 at 10 18 08 PM

Screen Shot 2021-01-18 at 10 18 20 PM

Screen Shot 2021-01-18 at 10 20 36 PM

Screen Shot 2021-01-18 at 10 18 50 PM

Activity

  1. tas50 commented on Jan 19, 2021

    @tas50
    Contributor

    I just came here to report this one. It's going to cause a lot of issues since iD is suggesting updating actual UPS mailboxes to be UPS Store mailboxes.

  2. bhousel commented on Jan 19, 2021

    @bhousel
    MemberAuthor

    I did this! Template items now support include/exclude patterns that will match against the ids.
    I added an exclude for the "upsstore" post office and created an item for "UPS" operated mail boxes:

    Screen Shot 2021-01-19 at 5 20 14 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions