Adding Russian translation for UI by puzzle-rpa-team · Pull Request #63450 · apache/airflow

puzzle-rpa-team · 2026-03-12T14:34:34Z

Russian UI translation was added in this PR. Unfortunately, I am not an Airflow contributor, so I cannot add a code owner to CODEOWNERS; maybe someone would be willing to step in?
Thank you to all reviewers for their time in advance!

Was generative AI tooling used to co-author this PR?

Yes (please specify the tool below)

Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
When adding dependency, check compliance with the ASF 3rd Party License Policy.
For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

boring-cyborg · 2026-03-12T14:34:38Z

Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
Here are some useful points:

Pay attention to the quality of your code (ruff, mypy and type annotations). Our prek-hooks will help you with that.
In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example DAG that shows how users should use it.
Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
Be sure to read the Airflow Coding style.
Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
Apache Airflow is a community-driven project and together we are making it better 🚀.
In case of doubts contact the developers at:
Mailing List: dev@airflow.apache.org
Slack: https://s.apache.org/airflow-slack

choo121600

Wow, adding a new language! Thanks a lot for the contribution 🙌

choo121600 · 2026-03-12T14:43:59Z

nit: It looks like there might still be some untranslated or unused strings.

The following command might help identify and fix them:

breeze ui check-translation-completeness --language ru --add-missing --remove-unused

pierrejeambrun

Cool! A new translation 🎉

Can't vouch for the translation quality, but as per our policy this will need a translation owner, and probably a sponsor.

More information in public/i18n/README.md. Would you be willing to become a translation owner, if yes do you know if someone in the community with 'write' access to the repo would be willing to sponsor you?

Otherwise you can reach out to the mailing list to find a trusted party in the community that is willing to take ownership for rusian translation.

Code owners file need to be updated.

puzzle-rpa-team · 2026-03-12T15:12:12Z

I'm willing to become a translation owner if there isn't somebody already working on a Russian translation, but to my knowledge no one left a PR similar to mine, or started a Slack discussion on the topic. I guess I'll just ask on Slack?
It's my first contribution so I don't know anyone in the community (I'll update the CODEOWNERS file as per section 6.1 of public/i18n/README.md once I know more)

puzzle-rpa-team · 2026-03-13T07:25:12Z

I updated CODEOWNERS to include jscheffl as a code owner, myself as a translation owner and renat-sagut as an engaged translator since they made quite a few helpful contributions and corrections

.github/CODEOWNERS

jscheffl · 2026-03-13T19:13:28Z

Note: CI checks are failing but this looks rather like inherited from problems on main. I assume on next push this will turn green.

renat-sagut · 2026-03-14T22:31:39Z

In general, I have some doubts that "Даг" is a good translation for DAG in Russian. I skimmed a few articles about Airflow in Russian, and most of them seem to keep the term "DAG" in English rather than transliterating it.

@puzzle-rpa-team should we follow the same approach here and keep "DAG" as is? What do you think?

https://practicum.yandex.ru/blog/apache-airflow/
https://habr.com/ru/articles/811807/
https://bigdataschool.ru/blog/news/apache-airflow-architecture-docker-setup/
https://gitverse.ru/blog/articles/architecture/64-apache-airflow-chto-eto-takoe-i-principy-raboty-arhitektury

jscheffl · 2026-03-14T22:32:47Z

@puzzle-rpa-team the static checks failed, as I was on it anyway I pushed some fixes. Next time please check with contribution guides and install pre-commit checks with prek.
Fixes were made by prek run -a ts-compile-lint-ui and prek run -a update-breeze-cmd-output.

renat-sagut

a few more findings

airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json

# Conflicts: # .github/CODEOWNERS # airflow-core/src/airflow/ui/public/i18n/locales/ru/assets.json # airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json # airflow-core/src/airflow/ui/public/i18n/locales/ru/dashboard.json

puzzle-rpa-team · 2026-03-16T07:24:39Z

Added "few" and "many" forms to the translation, also edited CODEOWNERS (now it's airflow-core/src/airflow/ui/public/i18n/locales/ru/ @jscheffl # + @puzzle-rpa-team @renat-sagut)

In general, I have some doubts that "Даг" is a good translation for DAG in Russian. I skimmed a few articles about Airflow in Russian, and most of them seem to keep the term "DAG" in English rather than transliterating it.

@puzzle-rpa-team should we follow the same approach here and keep "DAG" as is? What do you think?

Good point, but I tried to keep as much of the translation Cyrillic as possible - I thought about translating "Dag" as "НАГ", even, to keep a "directed acyclic graph" acronym. But that sounded too weird and I settled on transliterating instead.
In my last commit I changed all instances of "Даг" to "ДАГ", if that's an acceptable middle ground.

jscheffl

Cool! That looks promising!

Unfortunately there was some instability with files dev/breeze/doc/images/output_pr_auto-triage.* on main which forced you to add them to PR... and as this was fixed they (sorry) need to be re-generated on this PR (As the new language is listed there as one CLI option)

I am not an expert with Cyrillic letters... you now wrote you used "DAG" in Cyrillic letters? Correct understood?
As a community (need to find the email discussion thread if needed) we decided to refer to the term "DAG" in UI as using "Dag" (just one upper case letter) and keep "DAG" only in the class of the code (no need to migrate all Dags). Not sure if the comment by @renat-sagut referred to "Dag" or the translation that was in before. Cyrillic "Dag" would be more correct than "DAG".

The language keys now look great!

Approving... once the conflict is resolved from my side good to be merged (with above being a sanity question)

renat-sagut · 2026-03-16T19:58:48Z

It looks like _few and _many are also missing in several other places. For example, favoriteDags_many and assetEvent_few, assetEvent_many are missing on the home page.

renat-sagut

a few more suggestions

airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json

airflow-core/src/airflow/ui/public/i18n/locales/ru/dag.json

airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json

airflow-core/src/airflow/ui/public/i18n/locales/ru/admin.json

Co-authored-by: renat-sagut <renat.sagut@gmail.com>

boring-cyborg · 2026-03-17T21:34:42Z

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

github-actions · 2026-03-17T21:36:25Z

Backport failed to create: v3-1-test. View the failure log Run details

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status	Branch	Result
❌	v3-1-test

You can attempt to backport this manually by running:

cherry_picker f0018f8 v3-1-test

This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

If you don't have cherry-picker installed, see the installation guide.

potiuk · 2026-03-18T07:13:03Z

NICE !

choo121600 · 2026-03-18T08:28:58Z

Cool! Congratulations on your first contribution and Welcome 🎉

* feat: added russian ui translation * fix: missing "one" and "many" for "files" * fix: added "_few" * fix: removed unused "few"'s, added "asset_many" * Update airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json Co-authored-by: renat-sagut <renat.sagut@gmail.com> * Update airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json Co-authored-by: renat-sagut <renat.sagut@gmail.com> * Apply suggestions from code review Co-authored-by: renat-sagut <renat.sagut@gmail.com> * CODEOWNERS updated to include locales/ru * Update .github/CODEOWNERS * Fix static checks * CODEOWNERS fixed + replaced "Даг" with "DAG" * Apply suggestions from code review Co-authored-by: renat-sagut <renat.sagut@gmail.com> * Apply suggestions from code review Co-authored-by: renat-sagut <renat.sagut@gmail.com> * misc translation fixes * Fix ts-compile-lint-ui --------- Co-authored-by: o.marchuk <o.marchuk@mkskom.ru> Co-authored-by: renat-sagut <renat.sagut@gmail.com> Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com> Co-authored-by: Jens Scheffler <jscheffl@apache.org>

o.marchuk and others added 4 commits March 12, 2026 14:35

feat: added russian ui translation

13c7d4e

fix: missing "one" and "many" for "files"

445ab40

fix: added "_few"

aedf97e

Merge branch 'apache:main' into main

534f718

puzzle-rpa-team requested review from amoghrajesh, ashb, bbovenzi, bugraoz93, choo121600, ephraimbuddy, gopidesupavan, guan404ming, jason810496, jedcunningham, jscheffl, pierrejeambrun, potiuk, ryanahamilton, shubhamraj-git and vatsrahul1001 as code owners March 12, 2026 14:34

boring-cyborg bot added area:dev-tools area:translations area:UI Related to UI/UX. For Frontend Developers. backport-to-v3-1-test labels Mar 12, 2026

choo121600 reviewed Mar 12, 2026

View reviewed changes

fix: removed unused "few"'s, added "asset_many"

3effb70

pierrejeambrun reviewed Mar 12, 2026

View reviewed changes

jscheffl reviewed Mar 13, 2026

View reviewed changes

.github/CODEOWNERS Outdated Show resolved Hide resolved

jscheffl and others added 3 commits March 14, 2026 22:28

Update .github/CODEOWNERS

0c41cbc

Merge branch 'main' into main

0fdbac1

Fix static checks

57162de

renat-sagut reviewed Mar 14, 2026

View reviewed changes

airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json Outdated Show resolved Hide resolved

airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json Outdated Show resolved Hide resolved

o.marchuk added 2 commits March 16, 2026 10:06

CODEOWNERS fixed + replaced "Даг" with "DAG"

3540460

Merge remote-tracking branch 'origin/main'

ad4ec9e

# Conflicts: # .github/CODEOWNERS # airflow-core/src/airflow/ui/public/i18n/locales/ru/assets.json # airflow-core/src/airflow/ui/public/i18n/locales/ru/common.json # airflow-core/src/airflow/ui/public/i18n/locales/ru/dashboard.json

jscheffl approved these changes Mar 16, 2026

View reviewed changes

renat-sagut reviewed Mar 16, 2026

View reviewed changes

puzzle-rpa-team and others added 5 commits March 17, 2026 10:26

Apply suggestions from code review

4dd7ad0

Co-authored-by: renat-sagut <renat.sagut@gmail.com>

Apply suggestions from code review

c9d9d29

Co-authored-by: renat-sagut <renat.sagut@gmail.com>

misc translation fixes

a841ef1

Merge remote-tracking branch 'upstream/main' into puzzle-rpa-team/main

d820794

Fix ts-compile-lint-ui

b3c3da6

jscheffl merged commit f0018f8 into apache:main Mar 17, 2026
134 checks passed

Conversation

puzzle-rpa-team commented Mar 12, 2026

Was generative AI tooling used to co-author this PR?

Uh oh!

boring-cyborg bot commented Mar 12, 2026

Uh oh!

choo121600 left a comment

Choose a reason for hiding this comment

Uh oh!

choo121600 commented Mar 12, 2026

Uh oh!

pierrejeambrun left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

puzzle-rpa-team commented Mar 12, 2026

Uh oh!

puzzle-rpa-team commented Mar 13, 2026

Uh oh!

Uh oh!

jscheffl commented Mar 13, 2026

Uh oh!

renat-sagut commented Mar 14, 2026

Uh oh!

jscheffl commented Mar 14, 2026

Uh oh!

renat-sagut left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

puzzle-rpa-team commented Mar 16, 2026

Uh oh!

jscheffl left a comment

Choose a reason for hiding this comment

Uh oh!

renat-sagut commented Mar 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

renat-sagut left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

boring-cyborg bot commented Mar 17, 2026

Uh oh!

github-actions bot commented Mar 17, 2026

Backport failed to create: v3-1-test. View the failure log Run details

Uh oh!

potiuk commented Mar 18, 2026

Uh oh!

choo121600 commented Mar 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

pierrejeambrun left a comment •

edited

Loading

renat-sagut commented Mar 16, 2026 •

edited

Loading