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

Resolving the Warning Issue #3775

Closed
chankailong opened this issue Apr 28, 2020 · 2 comments
Closed

Resolving the Warning Issue #3775

chankailong opened this issue Apr 28, 2020 · 2 comments

Comments

@chankailong
Copy link

"amenity/payment_terminal|ПриватБанк" -> duplicates? -> "amenity/bank|ПриватБанк"
"amenity/payment_terminal|Сбербанк" -> duplicates? -> "amenity/bank|Сбербанк"

After reading the wiki, I suggested to add ""matchTags": ["amenity/payment_terminal"]" to "amenity/bank".

https://www.wikidata.org/wiki/Q1515015
https://www.wikidata.org/wiki/Q205012

If it's fine, I can help to fix

@bhousel
Copy link
Member

bhousel commented Apr 28, 2020

For this one, I think people are mapping banks separate from payment terminals, so adding matchTags is probably not what we want to do..

I think we could treat the payment terminals like ATMs, but we aren’t really sure what to do with ATMs yet either (#2883 and #3056)

@Adamant36 Adamant36 added the considering Not Actionable - still considering if this is something we want label May 17, 2020
@matkoniecz
Copy link
Contributor

I will close this one as it is OK to map payment terminals. Thanks for being interested, but this specific idea would have unexpected consequences.

@matkoniecz matkoniecz removed the considering Not Actionable - still considering if this is something we want label Jul 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants