Skip to content

chore(fe): update Ukrainian translations#3872

Open
sea-snake-translation-bot wants to merge 2 commits into
dfinity:mainfrom
sea-snake-translation-bot:chore/translate-uk
Open

chore(fe): update Ukrainian translations#3872
sea-snake-translation-bot wants to merge 2 commits into
dfinity:mainfrom
sea-snake-translation-bot:chore/translate-uk

Conversation

@sea-snake-translation-bot
Copy link
Copy Markdown
Contributor

New translations were missing for `uk`. This PR adds the missing translations.

Changes

  • Translated missing entries in `src/frontend/src/lib/locales/uk.po`

Copilot AI review requested due to automatic review settings May 8, 2026 21:44
@sea-snake-translation-bot sea-snake-translation-bot requested a review from a team as a code owner May 8, 2026 21:44
Copy link
Copy Markdown
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

This PR aims to fill in missing Ukrainian (uk) frontend translations by updating the PO catalog used by the app’s i18n layer.

Changes:

  • Added/updated Ukrainian PO entries related to SSO/OpenID configuration error messaging.
  • Removed a couple of existing SSO-related Ukrainian entries.

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

Comment thread src/frontend/src/lib/locales/uk.po Outdated
Comment on lines +34 to +35
msgid "{domainInput} took too long to respond. Try again in a moment."
msgstr "{domainInput} занадто довго не відповідав. Спробуйте ще раз за мить."
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Removed the entry for {domainInput} took too long to respond. Try again in a moment. from uk.po — this msgid has no corresponding entry in en.po and would never be used.

Comment thread src/frontend/src/lib/locales/uk.po Outdated
Comment on lines +283 to +284
msgid "Couldn't load SSO settings from {domainInput}. Ask your SSO admin to check that /.well-known/ii-openid-configuration is reachable."
msgstr "Не вдалося завантажити налаштування SSO з {domainInput}. Попросіть адміністратора SSO перевірити, чи доступний /.well-known/ii-openid-configuration."
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Reverted the msgid back to "Couldn't reach {domainInput}. Check the spelling and your network." to match en.po, and restored the original Ukrainian msgstr.


msgid "Set as default sign-in"
msgstr "Встановити як вхід за замовчуванням"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Re-added the entry for "Several SSO sign-ins are in flight already. Wait a moment." with the original Ukrainian translation.


msgid "To finish setting up your passkey, follow the instructions shown on your <0>new device</0>."
msgstr "Щоб завершити налаштування вашого ключа доступу, дотримуйтесь інструкцій, показаних на вашому <0>новому пристрої</0>."

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Re-added the entry for "Too many recent attempts for {domainInput}. Wait a few minutes." with the original Ukrainian translation.

- Remove entry for "{domainInput} took too long to respond..." which has no matching msgid in en.po
- Revert msgid "Couldn't load SSO settings..." back to "Couldn't reach {domainInput}..." to match en.po
- Re-add missing entries for "Several SSO sign-ins are in flight already..." and "Too many recent attempts for {domainInput}..."
@sea-snake-translation-bot
Copy link
Copy Markdown
Contributor Author

Thanks for the review summary, @Copilot. All four inline issues (orphaned msgid, mismatched msgid key, and the two removed entries) have been addressed in the latest commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants