Skip to content

Conversation

@0rzech
Copy link
Contributor

@0rzech 0rzech commented Aug 21, 2025

Important

Add missing Polish translations to assistants.json, common.json, hub.json, and settings.json.

  • Translations:
    • Added missing Polish translations in assistants.json for instructionsDateHint.
    • Added translations in common.json for modelValidationStarted, modelValidationFailed, and downloadAndVerificationComplete.
    • Added translation in hub.json for tools.
    • Added translations in settings.json for startupConfiguration, runOnStartup, and runOnStartupDesc.

This description was created by Ellipsis for f08a639. You can customize this summary. It will automatically update as commits are pushed.

@0rzech 0rzech force-pushed the fix/add-missing-polish-translations branch from f08a639 to 1a35da5 Compare August 21, 2025 10:29
@0rzech 0rzech changed the title Add missing Polish translations fix: add missing Polish translations Aug 21, 2025
@0rzech 0rzech force-pushed the fix/add-missing-polish-translations branch 2 times, most recently from 1a35da5 to f6cda23 Compare August 21, 2025 10:30
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to f08a639 in 1 minute and 43 seconds. Click for details.
  • Reviewed 60 lines of code in 4 files
  • Skipped 0 files when reviewing.
  • Skipped posting 4 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. web-app/src/locales/pl/assistants.json:33
  • Draft comment:
    Translation for 'instructionsDateHint' is correctly added.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. web-app/src/locales/pl/common.json:263
  • Draft comment:
    New model validation and download completion translations look good. Verify that any special whitespace (e.g. non-breaking space) in "został pobrany" is intentional.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
3. web-app/src/locales/pl/hub.json:15
  • Draft comment:
    Added 'tools' translation as 'Narzędzia' is consistent with other locale files.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
4. web-app/src/locales/pl/settings.json:164
  • Draft comment:
    Startup configuration translations are clear and properly added.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_3PVf7iOEctRuXmeu

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@louis-jan
Copy link
Contributor

Thank you so much! @0rzech

@0rzech 0rzech force-pushed the fix/add-missing-polish-translations branch from f6cda23 to c8b8b7c Compare August 21, 2025 10:33
Copy link
Contributor

@louis-jan louis-jan left a comment

Choose a reason for hiding this comment

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

LGTM

@louis-jan louis-jan merged commit ef90f07 into janhq:dev Aug 21, 2025
16 checks passed
@github-project-automation github-project-automation bot moved this to QA in Jan Aug 21, 2025
@0rzech 0rzech deleted the fix/add-missing-polish-translations branch August 21, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

2 participants