You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "renat-sagut (via GitHub)" <gi...@apache.org> on 2026/05/17 20:50:41 UTC
[PR] Add missing russian translations [airflow]
renat-sagut opened a new pull request, #67073:
URL: https://github.com/apache/airflow/pull/67073
<!-- SPDX-License-Identifier: Apache-2.0
https://www.apache.org/licenses/LICENSE-2.0 -->
<!--
Thank you for contributing!
Please provide above a brief description of the changes made in this pull request.
Write a good git commit message following this guide: http://chris.beams.io/posts/git-commit/
Please make sure that your code changes are covered with tests.
And in case of new features or big changes remember to adjust the documentation.
Feel free to ping (in general) for the review if you do not see reaction for a few days
(72 Hours is the minimum reaction time you can expect from volunteers) - we sometimes miss notifications.
In case of an existing issue, reference it using one of the following:
* closes: #ISSUE
* related: #ISSUE
-->
* Added missing russian translations
---
##### Was generative AI tooling used to co-author this PR?
<!--
If generative AI tooling has been used in the process of authoring this PR, please
change below checkbox to `[X]` followed by the name of the tool, uncomment the "Generated-by".
-->
- [ ] Yes (please specify the tool below)
<!--
Generated-by: [Tool Name] following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
-->
---
* Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#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](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals)) is needed.
* When adding dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
* For significant user-facing changes create newsfragment: `{pr_number}.significant.rst`, in [airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments). You can add this file in a follow-up commit after the PR is created so you know the PR number.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add missing russian translations [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #67073:
URL: https://github.com/apache/airflow/pull/67073#issuecomment-4472766240
some errors :(
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add missing russian translations [airflow]
Posted by "bbovenzi (via GitHub)" <gi...@apache.org>.
bbovenzi closed pull request #67073: Add missing russian translations
URL: https://github.com/apache/airflow/pull/67073
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add missing russian translations [airflow]
Posted by "puzzle-rpa-team (via GitHub)" <gi...@apache.org>.
puzzle-rpa-team commented on PR #67073:
URL: https://github.com/apache/airflow/pull/67073#issuecomment-4853369311
> "real" technical approval. @puzzle-rpa-team are you able to review?
Sorry for late reply, is it possible to reopen this PR? Changes @renat-sagut made mostly cover the last plural form I didn't add before here: https://github.com/apache/airflow/pull/63450, and some miscellaneous translation fixes
What do I do as a translation owner? Should I take over this PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add missing russian translations [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk closed pull request #67073: Add missing russian translations
URL: https://github.com/apache/airflow/pull/67073
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add missing russian translations [airflow]
Posted by "bbovenzi (via GitHub)" <gi...@apache.org>.
bbovenzi commented on PR #67073:
URL: https://github.com/apache/airflow/pull/67073#issuecomment-6046357287
@puzzle-rpa-team or @renat-sagut this PR is quite stale. I'd recommend opening a new PR against the latest on main
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add missing russian translations [airflow]
Posted by "Copilot (via GitHub)" <gi...@apache.org>.
Copilot commented on code in PR #67073:
URL: https://github.com/apache/airflow/pull/67073#discussion_r3401252440
##########
airflow-core/src/airflow/ui/public/i18n/locales/ru/hitl.json:
##########
@@ -8,9 +8,12 @@
"received": "Просмотрено"
}
},
- "requiredAction_one": "Необходимое действие",
- "requiredAction_other": "Необходимые действия",
+ "requiredAction_few": "{{count}} обязательных действия",
+ "requiredAction_many": "{{count}} обязательных действий",
+ "requiredAction_one": "{{count}} обязательное действие",
+ "requiredAction_other": "{{count}} обязательных действия",
Review Comment:
`requiredAction_*` strings now interpolate `{{count}}`, but the UI renders `requiredAction_other` as a label without providing `count` (e.g. in `NeedsReviewButton`), which will show the raw `{{count}}` placeholder to users. These label strings should not require interpolation; the count-specific text belongs in `requiredActionCount_*`.
##########
airflow-core/src/airflow/ui/public/i18n/locales/ru/admin.json:
##########
@@ -20,14 +20,19 @@
"host": "Хост",
"port": "Порт"
},
+ "connection_few": "Соединения",
+ "connection_many": "Соединений",
"connection_one": "Соединение",
- "connection_other": "Соединений",
+ "connection_other": "Соединения",
Review Comment:
`connections.connection_other` is used as a generic plural resource name (without passing `count`) in bulk toaster messages that already prepend `{{count}}`. With the current value ("Соединения"), messages like "5 Соединения ..." become grammatically incorrect; using genitive plural here avoids obviously wrong output for many counts.
##########
airflow-core/src/airflow/ui/public/i18n/locales/ru/admin.json:
##########
@@ -183,7 +197,9 @@
},
"noRowsMessage": "Переменные не найдены",
"searchPlaceholder": "Поиск ключей",
+ "variable_few": "Переменные",
+ "variable_many": "Переменных",
"variable_one": "Переменная",
- "variable_other": "Переменные"
+ "variable_other": "Переменной"
Review Comment:
`variables.variable_other` is used as a generic plural resource name (without passing `count`) in bulk toaster messages that already prepend `{{count}}`. The current value ("Переменной") is singular, leading to incorrect UI text like "3 Переменной ..."; it should be a plural form suitable for bulk operations.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add missing russian translations [airflow]
Posted by "jscheffl (via GitHub)" <gi...@apache.org>.
jscheffl commented on PR #67073:
URL: https://github.com/apache/airflow/pull/67073#issuecomment-4688144695
@renat-sagut seems some translation changes have been made in paralel, can you rebase with main? Also since a month there might have been some changes /extensions in parallel
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org