-
-
Notifications
You must be signed in to change notification settings - Fork 126
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
Russian localisation added #308
Conversation
Reviewer's Guide by SourceryThis pull request adds Russian localization to the project by introducing new JSON files containing translated strings for various features and components of the application. File-Level Changes
Tips
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @eremeyko - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 2 issues found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.
@@ -0,0 +1,4 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue (typo): Incorrect language name and flag for Russian localization
{ | ||
"no_result": "К сожалению, не удалось найти никаких результатов!", | ||
"no_result_w_query": "К сожалению, нет результатов по запросу: {kueri}", | ||
"get_data": "⏳ Пожалуйста, подождите, получаю данные..", | ||
"cl_btn": "❌ Закрыть", | ||
"back_btn": "↩️ Вернуться", | ||
"dl_text": "⬇️ Скачать", | ||
"cat_text": "Категория", | ||
"quality": "Качество", | ||
"ex_data": "👇 Вытащить данные ", | ||
"unauth": "Эта кнопка не для Вас..", | ||
"invalid_cb": "Неверные данные обратного вызова, отправьте команду еще раз..", | ||
"res_scrape": "<b>Собранная информация с</b> <code>{link}</code>:\n\n{kl}", | ||
"header_with_query": "<b>#{web} Результаты Для:</b> <code>{kueri}</code>\n\n", | ||
"header_no_query": "<b>#{web} Последнее:</b>\n🌀 Используйте /{cmd} [заголовок] чтобы начать поиск по заголовку.\n\n", | ||
"invalid_cmd_scrape": "Используйте команду /{cmd} <b>[link]</b> для поиска ссылки на скачивание.", | ||
"err_getweb": "ОШИБКА: Не удалось получить данные с интернета из-за {err}.", | ||
"err_getapi": "ОШИБКА: Не удалось получить данные с АПИ", | ||
"unsupport_dl_btn": "Некоторые результаты не отображаются в кнопке извлечения из-за неподдерживаемой ссылки." | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: Consider centralizing common phrases to reduce duplication
{
"common": {
"no_result": "К сожалению, не удалось найти никаких результатов!",
"wait": "⏳ Пожалуйста, подождите, получаю данные..",
"buttons": {
"close": "❌ Закрыть",
"back": "↩️ Вернуться",
"download": "⬇️ Скачать",
"extract": "👇 Вытащить данные "
}
},
"no_result_w_query": "К сожалению, нет результатов по запросу: {kueri}",
"cat_text": "Категория",
"quality": "Качество",
"unauth": "Эта кнопка не для Вас..",
"invalid_cb": "Неверные данные обратного вызова, отправьте команду еще раз.."
}
Summary by Sourcery
Introduce Russian localization by adding a set of JSON files with Russian translations for different modules, enhancing the application's multilingual capabilities.
New Features: