-
Notifications
You must be signed in to change notification settings - Fork 842
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
adding 'brand' data from Dohánybolt #2009
Conversation
config/canonical.json
Outdated
"tags": { | ||
"brand": "Nemzeti Dohánybolt", | ||
"brand:wikidata": "Q20639040", | ||
"brand:wikipedia": "en:Dohánybolt", | ||
"countryCodes": ["hu"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has to be below the count
tag...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the "countryCode" tag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this does not belong into the "tags" object... it should look like this:
"shop/tobacco|Nemzeti Dohánybolt": {
"count": 877,
"countryCodes": ["hu"],
"match": [
"shop/tobacco|Dohánybolt",
"shop/tobacco|Nemzeti dohánybolt"
],
"tags": {
"brand": "Nemzeti Dohánybolt",
"brand:wikidata": "Q20639040",
"brand:wikipedia": "en:Dohánybolt",
"name": "Nemzeti Dohánybolt",
"shop": "tobacco"
}
},
BTW: Google translates it directly to tobacco shop (https://translate.google.de/#auto/en/doh%C3%A1nybolt) Are you sure that this is no generic name after all? |
It is kind of generic. Expect for the fact that they're all state run, so the same actor is behind them, just like a commerical brand. However, this is probably worth a bigger debate |
This is old, so I'm just going to do what @ImreSamu suggested here: |
(closes #1811, closes #2009) (see #2028 (comment))
fixes #1811
source