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

Ballet studios / dancing schools #1177

Closed
FloEdelmann opened this issue Mar 23, 2024 · 5 comments
Closed

Ballet studios / dancing schools #1177

FloEdelmann opened this issue Mar 23, 2024 · 5 comments
Labels
bug Something isn't working translations this is an issue with a translation, which needs to be solved on the transifex platform

Comments

@FloEdelmann
Copy link
Contributor

I searched for "Ballettstudio" (in German) and nothing was found. In a forum thread I found that I should rather search for dancing school ("Tanzschule" in German) and add ballet as a dancing style there. This worked. So first suggestion: Can we add "ballet studio" as a term for the Dancing School preset?

However, the Dancing School preset (amenity = dancing_school) was marked as deprecated and suggested replacing it with the Dance Hall preset preset (leisure = dance + dance:teaching = yes), which was fine for me. It seems wrong to have a suggested preset that is marked as deprecated though. So second suggestion: Should we remove the Dancing School preset and add its terms to the Dance Hall preset?

Note also that there is an issue about a wrong deprecation of amenity = dancing_school:

So maybe the deprecation is wrong at all?

@simonpoole
Copy link

This is a leftover of the previous reign of maintainers were the policy was "we tag everything in our myopic view and basta". There is unluckily still a lot of legacy from that period left (for example office=coworking).

@bhousel
Copy link
Member

bhousel commented Mar 25, 2024

This is a leftover of the previous reign of maintainers were the policy was "we tag everything in our myopic view and basta". There is unluckily still a lot of legacy from that period left (for example office=coworking).

It's really uncool of you to write stuff like this. Be better, Simon.

For the curious, here's a link to the thing you are still mad about: openstreetmap/iD#6251

TLDR - 5 years ago, @westnordost sent me a PR to deprecate a thing based on what the wiki said, and I said "sure". Somehow that makes me a tag tyrant in Simon's view.

@simonpoole
Copy link

That was not about you falling indirectly for a notorious wiki fiddler, but you refusing to fix the error once it was pointed out to you.

@tyrasd
Copy link
Member

tyrasd commented Apr 5, 2024

@FloEdelmann

second suggestion

Yeah, there should never be a preset for a tag which is also in the "deprecated" tags list.

However, our intention was to have both the Dancing School and Dance Hall presets to be valid, proper presets of related, but slightly different map features. I had thought that this was already fixed with #814, but it looks like we had accidentally change the wrong file (the dist folder is overwritten by the build scripts, so that PR had little effect) 🙈 915ed76 implements this now properly. Sorry for the oversight 🙇 and thanks for reporting this discrepancy!

@tyrasd tyrasd added bug Something isn't working translations this is an issue with a translation, which needs to be solved on the transifex platform labels Apr 5, 2024
@tyrasd
Copy link
Member

tyrasd commented Apr 5, 2024

I searched for "Ballettstudio" (in German) and nothing was found.

This is most likely just missing in the German translations of the search terms of this preset. However, translations are not managed on github, but on a dedicated platform called transifex (see https://github.com/openstreetmap/id-tagging-schema/blob/main/CONTRIBUTING.md#Translating). You should be able to fix the German strings on there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working translations this is an issue with a translation, which needs to be solved on the transifex platform
Projects
None yet
Development

No branches or pull requests

4 participants