organicmaps / organicmaps

🍃 Organic Maps is a free Android & iOS offline maps app for travelers, tourists, hikers, and cyclists. It uses crowd-sourced OpenStreetMap data and is developed with love by MapsWithMe (MapsMe) founders and our community. No ads, no tracking, no data collection, no crapware. Please donate to support the development!
https://organicmaps.app
Apache License 2.0
9.8k stars 941 forks source link

Switch to Weblate for localization #946

Closed rtsisyk closed 2 years ago

rtsisyk commented 3 years ago

https://github.com/WeblateOrg/weblate/issues/6346

biodranik commented 3 years ago

Каким ты видишь workflow? Сценарии:

  1. Разработчик вставляет новые строчки в PR. Что делать с weblate?
  2. Пользователи со временем напереводили в weblate. Как, когда и кем это вливается в master?
rtsisyk commented 3 years ago
  1. Weblate должен синкать в том числе и из репы. Из нагенеренных строк, естестевенно. Будем это проверять.
  2. Оно создает PR при наличии обновлений каждый n часов (настраиваемо) или по клику мышкой в интерфейсе. Причем в PR можно сохранить авторов переводов и добавить Signed-off-by.

https://github.com/organicmaps/organicmaps/pull/959 https://github.com/organicmaps/organicmaps/pull/958

Сейчас там есть несколько проблем - с whitespaces в Андроиде (всё переформатило), с непонятной полной перегенераций в iOS (что-то недонастроено), с сообщениями в commit messages. Я задаю вопросы, пытаюсь доделать.

Если всё заработает, то twine и ко можно будет выкинуть и оставить только сгенеренные файлы. Дальше переводчики будут топать на weblate.org и добавлять локализации. Там есть ревью. Есть suggestions. Есть даже автоматический перевод. Если пропоагация перевода с ведра в ios и наоборот (не тестил пока). Периодически будем нажмать на кнопку и вливать PR.

On Fri, Jul 30, 2021 at 12:40 PM Alexander Borsuk @.***> wrote:

Каким ты видишь workflow? Сценарии:

  1. Разработчик вставляет новые строчки в PR. Что делать с weblate?
  2. Пользователи со временем напереводили в weblate. Как, когда и кем это вливается в master?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/organicmaps/organicmaps/issues/946#issuecomment-889772112, or unsubscribe https://github.com/notifications/unsubscribe-auth/AANXHDVOCJW4C2MZ5TIBPMDT2JXPLANCNFSM5BD3HCZA .

-- WBR, Roman Tsisyk @.***>

biodranik commented 3 years ago

Как ты оставишь только сгенеренные файлы, если одни переводы должны быть только под айос, а другие только под андроид? Вангую третьи - под какие-то ещё операционки в будущем.

rtsisyk commented 3 years ago

Не понял проблему. Прямо сейчас в репе лежат сгенеренныне переводы. Weblate может начать слать pr прямо в сгенеренные файлы. Надо только проверить, что они реально умеют шарить локализацию между платформами.

On Fri, Jul 30, 2021 at 12:56 PM Alexander Borsuk @.***> wrote:

Как ты оставишь только сгенеренные файлы, если одни переводы должны быть только под айос, а другие только под андроид? Вангую третьи - под какие-то ещё операционки в будущем.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/organicmaps/organicmaps/issues/946#issuecomment-889780958, or unsubscribe https://github.com/notifications/unsubscribe-auth/AANXHDRGHMXRAJZZOWGAMUTT2JZL7ANCNFSM5BD3HCZA .

-- WBR, Roman Tsisyk @.***>

biodranik commented 3 years ago

У нас сейчас единственная актуальная точка входа для переводов - strings.txt Ты предлагаешь сделать две точки? Оставить как есть для PR, и второй точкой делать результат с weblate? Как ты это будешь синкать?

Если хочешь сдвинуть точку на сгенеренные файлы, то что делать разрабу? Сразу вносить правки в xml и айосные файлы? А потом оттуда импортить обратно в weblate?

rtsisyk commented 2 years ago

@biodranik please discuss Twine format support with Weblate folks if you are desperately want to save it:

https://github.com/WeblateOrg/weblate/issues/6346 https://github.com/WeblateOrg/weblate/issues/6741

biodranik commented 2 years ago

s/desperately/conveniently/

rtsisyk commented 2 years ago

We are not doing this approach. Discussed offlined.