Skip to content
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.

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

Fixup entries for RBC #9913

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fixup entries for RBC #9913

wants to merge 4 commits into from

Conversation

wolfy1339
Copy link
Contributor

The main RBC brand does not exist in the US anymore

In 2012, RBC Bank USA's banking division for American clients was sold to PNC Financial Services1

There are some mistagged entries in the USA where there is a bank named Royal Bank which is different from RBC.

In the rest of the world, the banks tagged as RBC are most likely not banks in the sense that we tag them in OSM.
I can't find any evidence that there are bank branches operated elsewhere than Canada and the Carribean under the RBC brand name

@Snowysauce Snowysauce added the fix data Need to cleanup the data in OSM after this is released label Sep 7, 2024
@UKChris-osm
Copy link
Collaborator

Not being Canadian myself, I don't know how things work with regards to the French-Canadian language, but could the two entries be merged into one, using name:fr as well, which I noticed is present in all the entries when you select the Overpass Turbo link under RBC Banque Royale

@bhousel
Copy link
Member

bhousel commented Sep 7, 2024

I think in the past we have done separate entries for Quebec because their brands use French in the name tag.

@Cj-Malone
Copy link
Collaborator

I think in the past we have done separate entries for Quebec because their brands use French in the name tag.

Yeah, but I'd like to discuss this and the HK thing of both languages in name after OSM.org switches to vector tiles. It is mapping for the renderer.

@wolfy1339
Copy link
Contributor Author

It's not just the name tag, it's also the display name that gets translated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix data Need to cleanup the data in OSM after this is released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants