Skip to content

chore(fe): update Polish translations#3870

Open
sea-snake-translation-bot wants to merge 1 commit into
dfinity:mainfrom
sea-snake-translation-bot:chore/translate-pl
Open

chore(fe): update Polish translations#3870
sea-snake-translation-bot wants to merge 1 commit into
dfinity:mainfrom
sea-snake-translation-bot:chore/translate-pl

Conversation

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

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

Changes

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

Copilot AI review requested due to automatic review settings May 8, 2026 21:43
@sea-snake-translation-bot sea-snake-translation-bot requested a review from a team as a code owner May 8, 2026 21:43
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 update the Polish (pl) Lingui message catalog to cover missing frontend translations.

Changes:

  • Adds a new Polish translation entry for a domain timeout message.
  • Replaces one SSO-related error string entry.
  • Removes two existing SSO-related entries from the Polish catalog.

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

Comment on lines +34 to +36
msgid "{domainInput} took too long to respond. Try again in a moment."
msgstr "{domainInput} zbyt długo nie odpowiadał. Spróbuj ponownie za chwilę."

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.

Thank you for the review. I verified this by running npm run extract on the PR branch: the msgid {domainInput} took too long to respond. Try again in a moment. does exist in en.po (it was recently added to the source). The translation in pl.po is correct and has been retained.

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 "Nie można załadować ustawień SSO z {domainInput}. Poproś administratora SSO o sprawdzenie, czy /.well-known/ii-openid-configuration jest dostępny."
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.

Thank you for the review. I confirmed via npm run extract that Couldn't load SSO settings from {domainInput}. Ask your SSO admin to check that /.well-known/ii-openid-configuration is reachable. is the current msgid in en.po — it replaced the old Couldn't reach {domainInput}... key when the source string changed. The pl.po entry correctly tracks the new msgid with an accurate Polish translation.

Comment on lines 884 to 888
msgid "Set as default sign-in"
msgstr "Ustaw jako domyślne logowanie"

msgid "Several SSO sign-ins are in flight already. Wait a moment."
msgstr "Kilka logowań SSO jest już w toku. Poczekaj chwilę."

msgid "Share {label}"
msgstr "Udostępnij {label}"
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.

Thank you for the review. Running npm run extract confirms that Several SSO sign-ins are in flight already. Wait a moment. no longer exists in en.po — it has been removed from the source catalog. Removing it from pl.po is correct; keeping it would cause stale data in the Polish catalog.

@@ -1083,9 +1083,6 @@ msgstr "Aby kontynuować:"
msgid "To finish setting up your passkey, follow the instructions shown on your <0>new device</0>."
msgstr "Aby dokończyć konfigurowanie klucza dostępu, postępuj zgodnie z instrukcjami wyświetlanymi na <0>nowym urządzeniu</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.

Thank you for the review. Running npm run extract confirms that Too many recent attempts for {domainInput}. Wait a few minutes. no longer exists in en.po — it has been removed from the source catalog. Removing it from pl.po is correct; keeping it would leave the Polish catalog out of sync with the source.

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

Thank you for the automated review overview. All four inline comments have been addressed individually — the three catalog-sync concerns (new timeout entry, replaced SSO-error key, removed obsolete keys) were each explained with reference to the current en.po source catalog.

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