Skip to content

Translations update from Hosted Weblate#341

Merged
n-ce merged 2 commits into
n-ce:Reinstate-fallbackfrom
weblate:weblate-ytify-web
Sep 7, 2025
Merged

Translations update from Hosted Weblate#341
n-ce merged 2 commits into
n-ce:Reinstate-fallbackfrom
weblate:weblate-ytify-web

Conversation

@weblate
Copy link
Copy Markdown
Contributor

@weblate weblate commented Sep 6, 2025

Translations update from Hosted Weblate for ytify/web.

Current translation status:

Weblate translation status

Summary by CodeRabbit

  • New Features
    • Added a new Russian translation for a settings option to prefer JioSaavn for music. Users with the app set to Russian will now see a localized label for this preference in Settings.
  • Localization
    • Updated the Russian locale file to include the new key and formatting adjustments to support the added entry. No other translations were changed.

Yurt Page and others added 2 commits September 7, 2025 00:05
Currently translated at 100.0% (174 of 174 strings)

Translation: ytify/web
Translate-URL: https://hosted.weblate.org/projects/ytify/web/ru/
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 6, 2025

Walkthrough

Adds a new Russian localization key "settings_jiosaavn" to src/locales/ru.json and updates the preceding "settings_reload" entry to include a trailing comma. No other files or logic are changed.

Changes

Cohort / File(s) Summary of changes
Localization (RU)
src/locales/ru.json
Added key "settings_jiosaavn": "Предпочитать JioSaavn для Музыки"; updated "settings_reload" to include a trailing comma to accommodate the new entry.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

A nibble of text in Cyrillic delight,
I hop through keys by moonlit byte.
A comma placed, a string anew—
JioSaavn’s preference shining through.
With gentle paws I localize,
ru.json hums—surprise, surprise! 🐇✨

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/locales/ru.json (1)

109-109: Wrong language string (Spanish) in Russian locale

“HTTP transmisión en vivo” is Spanish. Replace with a Russian label, e.g. “HTTP потоковое вещание (HLS)”.

Apply this diff:

-    "settings_hls": "HTTP transmisión en vivo",
+    "settings_hls": "HTTP потоковое вещание (HLS)",
🧹 Nitpick comments (2)
src/locales/ru.json (2)

176-178: Polish wording and capitalization for consistency

  • Use “Перезагрузить страницу, чтобы применить изменения” (more natural).
  • Lowercase “музыки”: “Предпочитать JioSaavn для музыки”.

Apply this diff:

-    "settings_reload": "Перегрузить страницу для применения изменений",
-    "settings_jiosaavn": "Предпочитать JioSaavn для Музыки"
+    "settings_reload": "Перезагрузить страницу, чтобы применить изменения",
+    "settings_jiosaavn": "Предпочитать JioSaavn для музыки"

90-90: Tone/style consistency

Use noun phrase like elsewhere (“Слушать позже”) instead of polite imperative.

Apply this diff:

-    "collection_selector_listen_later": "послушайте это позже",
+    "collection_selector_listen_later": "Слушать позже",
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 64e68ed and 81d2dbe.

📒 Files selected for processing (1)
  • src/locales/ru.json (1 hunks)
🔇 Additional comments (1)
src/locales/ru.json (1)

153-154: Possible message/key mismatch

Key suggests authentication success/failure, but strings describe logout. Confirm intent and adjust.

Suggested fix:

-    "piped_success_auth": "Вы успешно вышли из своей учётной записи Piped.",
-    "piped_failed_auth": "Не удалось выйти успешно",
+    "piped_success_auth": "Аутентификация Piped выполнена успешно.",
+    "piped_failed_auth": "Не удалось выполнить аутентификацию",

Comment thread src/locales/ru.json
@@ -173,5 +173,6 @@
"settings_audio_quality": "Качество звука",
"settings_prefetch": "Предскачивание очереди",
"settings_enforce_piped": "Приниудительно Piped для проигрывания",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Typo and clarity

“Приниудительно” → “Принудительно”, and clarify phrasing.

Apply this diff:

-    "settings_enforce_piped": "Приниудительно Piped для проигрывания",
+    "settings_enforce_piped": "Принудительно использовать Piped для воспроизведения",
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"settings_enforce_piped": "Приниудительно Piped для проигрывания",
"settings_enforce_piped": "Принудительно использовать Piped для воспроизведения",
🤖 Prompt for AI Agents
In src/locales/ru.json around line 175, the value for "settings_enforce_piped"
contains a typo and unclear phrasing; change "Приниудительно Piped для
проигрывания" to a corrected and clearer Russian string such as "Принудительно
использовать Piped для воспроизведения" (replace the existing value only,
preserving the key and surrounding JSON formatting).

@n-ce n-ce changed the base branch from main to Reinstate-fallback September 7, 2025 08:04
@n-ce n-ce deleted the branch n-ce:Reinstate-fallback September 7, 2025 08:05
@n-ce n-ce closed this Sep 7, 2025
@n-ce n-ce reopened this Sep 7, 2025
@n-ce n-ce changed the base branch from Reinstate-fallback to main September 7, 2025 08:07
@n-ce n-ce changed the base branch from main to Reinstate-fallback September 7, 2025 08:07
@n-ce n-ce merged commit ab40d51 into n-ce:Reinstate-fallback Sep 7, 2025
2 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Sep 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants