Skip to content

i18n - translations#17443

Merged
FelixMalfait merged 2 commits intomainfrom
i18n
Jan 26, 2026
Merged

i18n - translations#17443
FelixMalfait merged 2 commits intomainfrom
i18n

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Created by Github action

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Jan 26, 2026

Greptile Overview

Greptile Summary

This PR adds new translation strings for front component widget validation across all supported locales.

Changes Made

  • Added 4 new translation strings related to front component widget validation (IDs: 3UUjIe, tBK3H8, /xPxQo, T3xkLf)
  • Updated source file references to include validate-front-component-configuration-type.util.ts and validate-front-component-flat-page-layout-widget-for-creation.util.ts
  • Generated English translations are complete and properly formatted
  • All other locale .po files have empty msgstr values awaiting translation by the community or translation service
  • All 31 generated TypeScript files (.ts) have been recompiled with the new translation structure

Translation Status

  • English (en.po): ✓ Complete with proper translations
  • All other locales: Pending translation (empty msgstr values)

This is a standard automated i18n update following the addition of new translatable strings in the codebase from PR #17440.

Confidence Score: 5/5

  • This PR is safe to merge with no issues detected
  • This is an automated translation extraction PR that follows the standard i18n workflow. All changes are mechanically generated from source code, with proper English translations and correctly formatted .po and .ts files. No functional code changes are included.
  • No files require special attention

Important Files Changed

Filename Overview
packages/twenty-server/src/engine/core-modules/i18n/locales/en.po Added 4 new translation strings for front component widget validation with proper English translations
packages/twenty-server/src/engine/core-modules/i18n/locales/fr-FR.po Added 4 new translation strings with empty msgstr values awaiting French translation
packages/twenty-server/src/engine/core-modules/i18n/locales/de-DE.po Added 4 new translation strings with empty msgstr values awaiting German translation
packages/twenty-server/src/engine/core-modules/i18n/locales/es-ES.po Added 4 new translation strings with empty msgstr values awaiting Spanish translation
packages/twenty-server/src/engine/core-modules/i18n/locales/generated/en.ts Auto-generated TypeScript file updated with 4 new translation IDs in minified JSON format
packages/twenty-server/src/engine/core-modules/i18n/locales/ja-JP.po Added 4 new translation strings with empty msgstr values awaiting Japanese translation

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Code as Source Code
    participant Extract as Translation Extractor
    participant EN as en.po (English)
    participant Other as Other .po files
    participant Compile as Compiler
    participant TS as Generated .ts files
    participant App as Application

    Dev->>Code: Add translatable strings with lingui macros
    Code->>Extract: Run translation extraction
    Extract->>EN: Extract new msgid strings with IDs
    EN->>EN: Add English msgstr translations
    Extract->>Other: Propagate new msgid to all locales
    Other->>Other: Set empty msgstr (awaiting translation)
    EN->>Compile: Compile .po to TypeScript
    Other->>Compile: Compile .po to TypeScript
    Compile->>TS: Generate minified JSON in .ts files
    App->>TS: Import translations at runtime
    TS->>App: Provide localized strings
Loading

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 62 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-TW.po">

<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/zh-TW.po:714">
P2: Provide a Traditional Chinese translation for the new front component widget configuration error so zh-TW users don’t see the English fallback.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment on lines +714 to +715
msgid "Configuration is required for front component widget"
msgstr ""
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Jan 26, 2026

Choose a reason for hiding this comment

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

P2: Provide a Traditional Chinese translation for the new front component widget configuration error so zh-TW users don’t see the English fallback.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-server/src/engine/core-modules/i18n/locales/zh-TW.po, line 714:

<comment>Provide a Traditional Chinese translation for the new front component widget configuration error so zh-TW users don’t see the English fallback.</comment>

<file context>
@@ -709,6 +709,16 @@ msgstr ""
 
+#. js-lingui-id: 3UUjIe
+#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-front-component-flat-page-layout-widget-for-creation.util.ts
+msgid "Configuration is required for front component widget"
+msgstr ""
+
</file context>
Suggested change
msgid "Configuration is required for front component widget"
msgstr ""
msgid "Configuration is required for front component widget"
msgstr "前端元件小工具需要設定。"
Fix with Cubic

@FelixMalfait FelixMalfait merged commit 90a496b into main Jan 26, 2026
10 checks passed
@FelixMalfait FelixMalfait deleted the i18n branch January 26, 2026 13:08
@github-actions
Copy link
Copy Markdown
Contributor Author

Fails
🚫

node failed.

Log

Details
�[31mError: �[39m SyntaxError: Unexpected token 'C', "Contributo"... is not valid JSON
    at JSON.parse (<anonymous>)
�[90m    at parseJSONFromBytes (node:internal/deps/undici/undici:4259:19)�[39m
�[90m    at successSteps (node:internal/deps/undici/undici:6882:27)�[39m
�[90m    at readAllBytes (node:internal/deps/undici/undici:5807:13)�[39m
�[90m    at process.processTicksAndRejections (node:internal/process/task_queues:103:5)�[39m
danger-results://tmp/danger-results-47d705b8.json

Generated by 🚫 dangerJS against 4c1e8c6

camilo-agudelo-uma pushed a commit to innovation-grupo-uma/twenty-uma that referenced this pull request Feb 2, 2026
Created by Github action

---------

Co-authored-by: github-actions <github-actions@twenty.com>
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.

1 participant