Skip to content

New Crowdin updates#138

Open
Hakanbaban53 wants to merge 9 commits intomasterfrom
l10n_master
Open

New Crowdin updates#138
Hakanbaban53 wants to merge 9 commits intomasterfrom
l10n_master

Conversation

@Hakanbaban53
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 8, 2026 20:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the en-US i18n strings related to Rclone update notifications/status, likely reflecting a revised “download vs install” flow.

Changes:

  • Removes modals.about.updating from the en-US base strings.
  • Updates notification.*.rcloneUpdateComplete wording to indicate the update was downloaded (not installed).
  • Adds new notification.*.rcloneUpdateInstalled strings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"rcloneUpdateAvailable": "Rclone Update Available",
"whatsNewRclone": "What's New in Rclone",
"updating": "Updating...",
"updateRclone": "Download Rclone Update",
Copy link

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

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

modals.about.updating was removed from the en-US base strings, but the frontend template still references it (see about-modal.component.html around the rclone update button). This will display a missing-translation key for en-US (likely the default language). Re-add this key (or update the template to use an existing key like modals.about.downloading/rcloneUpdateStarted).

Suggested change
"updateRclone": "Download Rclone Update",
"updateRclone": "Download Rclone Update",
"updating": "Updating Rclone...",

Copilot uses AI. Check for mistakes.
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