name= on amenity_parcel_locker

[coolultra1]

Hi,

It seems to me, some templates for parcel lockers include a name= tag, some don't. The tag is generally unneccessary as the name that's rendered on Carto, for instance is the brand= tag. How about removing all name= from amenity=parcel_lockers, then?

[hanchao]

The labeling logic has not changed, it still use name.
gravitystorm/openstreetmap-carto#4512 (review)
https://github.com/gravitystorm/openstreetmap-carto/blob/master/style/amenity-points.mss#L2970

[coolultra1]

Though, for some reason, brand is (also??) rendered https://www.openstreetmap.org/node/9071299691

[hanchao]

Though, for some reason, brand is (also??) rendered https://www.openstreetmap.org/node/9071299691

This is a good case. My mistake.
When importing data, use the brand first
https://github.com/gravitystorm/openstreetmap-carto/blob/master/project.mml#L1490

[kjonosm]

We could also add "preserveTags": ["^name"] to the general properties in the file, see e.g. bicycle_rental.json for reference.

[hanchao]

I added "preserveTags": ["^name"] to the amenity/parcel_locker in 5876cc4

This will allow the name tag as an initial value, but users are encouraged to change it, and NSI will not overwrite the custom names that people add.

Alse see the discussion by @bhousel in #4985