TheFive / osmbc

Open Street Map Blog Collector
MIT License
28 stars 11 forks source link

remove pt-pt-t #1022

Closed fredao closed 7 months ago

fredao commented 7 months ago

Because weekly's automatic translations procduce PT-t

grafik-buykjn51kty1xxfzcaki55jw8h

Strubbl commented 7 months ago

Could we also add PT-PT here: https://osmbc.openstreetmap.de/config/languageflags and reuse the PT icon?

fredao commented 7 months ago

Just as you like it! I don't want to keep expecting myself and the two Portuguese to keep hitting the backspace button every time I do an automatic translation.

TheFive commented 7 months ago
image

reason coud be, that - to have a correct deepl translation PT is replaces by PT-PT for translation:

But this is also done for EN, so does EN has the same problem ?

(screenshot comes from the basic osmbc config file on the server)

(that this does not occur on EN may be that EN is seldom the target of a translation with language picture)

i accept this as OSMBC error, because it is not using language names consistent

TheFive commented 7 months ago
image

Deepl os supporting PT PT-BR and PT-PT (ok supporting because Backward compatibility is not really a support)

fredao commented 7 months ago

Then we would nevertheless still need an icon @Strubbl

Strubbl commented 7 months ago

I try to do this by this evening

fredao commented 7 months ago

No stress, we've been putting up with it for a while now, so we'll be able to endure it for a few more days.

Strubbl commented 7 months ago

the language icons are already existing since September 2022. I do not know why they are not configured in osmbc 🤷 So i just did this. Should work now.

fredao commented 7 months ago

solved! - thanks! image