Skip to content

i18n - translations#17932

Merged
charlesBochet merged 2 commits intomainfrom
i18n
Feb 13, 2026
Merged

i18n - translations#17932
charlesBochet 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 Feb 13, 2026

Greptile Overview

Greptile Summary

This PR is an automated translation compilation that adds 8 new translation strings for fields widget validation and removes duplicate entries across all 31 supported locales.

Changes:

  • Added 8 new validation messages for fields widgets (configuration requirements, viewId validation, and configuration type checks)
  • Removed duplicate "Widget type FIELDS is not supported yet" entry that appeared twice in multiple files
  • Removed duplicate file location references for 11 widget type validation messages
  • All changes applied consistently across 31 locale files (.po) and their corresponding generated TypeScript files

Translation Status:

  • English translations are complete with proper error messages
  • All other locales have empty translations (msgstr "") awaiting localization work
  • This is standard practice for automated translation extraction - source strings are added first, then localized separately

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • This is an automated translation compilation PR created by GitHub Actions. All changes are mechanical additions of new translation keys and cleanup of duplicate entries. The changes follow the established .po file format correctly, and the generated TypeScript files are properly compiled. No logic changes or functional modifications are present.
  • No files require special attention

Important Files Changed

Filename Overview
packages/twenty-server/src/engine/core-modules/i18n/locales/en.po Added 8 new translation strings for fields widget validation and removed one duplicate "Widget type FIELDS" translation entry. Proper .po format maintained.
packages/twenty-server/src/engine/core-modules/i18n/locales/generated/en.ts Auto-generated TypeScript file compiled from .po source with new translation keys added. Single-line minified format is correct for generated code.
packages/twenty-server/src/engine/core-modules/i18n/locales/fr-FR.po Added same 8 new translation strings as English version but left translations empty (msgstr "") for localization. Removed duplicate entry like English.
packages/twenty-server/src/engine/core-modules/i18n/locales/generated/fr-FR.ts Auto-generated TypeScript file compiled from French .po source. Properly reflects empty translations for new strings awaiting localization.

Last reviewed commit: f0b6186

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.

62 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.

4 issues 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/tr-TR.po">

<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/tr-TR.po:748">
P2: New Turkish locale strings were added without translations (msgstr is empty), so users will see untranslated messages. Provide Turkish translations for the new msgids.</violation>
</file>

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

<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ko-KR.po:748">
P3: New Korean translations were added with empty msgstr values, which will cause ko-KR users to see English fallback strings. Please provide localized translations for these new messages.</violation>
</file>

<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:748">
P2: New zh-TW translations are added with empty msgstr values, leaving these user-facing errors untranslated. Provide localized strings for the new entries.</violation>
</file>

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

<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/ja-JP.po:748">
P2: New Japanese locale strings were added without translations (empty msgstr). This leaves the locale incomplete and causes these messages to display in English or as missing translations.</violation>
</file>

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

#. js-lingui-id: yU57/s
#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-fields-flat-page-layout-widget-for-creation.util.ts
msgid "Configuration is required for fields widget"
msgstr ""
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Feb 13, 2026

Choose a reason for hiding this comment

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

P2: New Turkish locale strings were added without translations (msgstr is empty), so users will see untranslated messages. Provide Turkish translations for the new msgids.

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/tr-TR.po, line 748:

<comment>New Turkish locale strings were added without translations (msgstr is empty), so users will see untranslated messages. Provide Turkish translations for the new msgids.</comment>

<file context>
@@ -742,6 +742,16 @@ msgstr ""
+#. js-lingui-id: yU57/s
+#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-fields-flat-page-layout-widget-for-creation.util.ts
+msgid "Configuration is required for fields widget"
+msgstr ""
+
+#. js-lingui-id: tj1Qu0
</file context>
Fix with Cubic

#. js-lingui-id: yU57/s
#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-fields-flat-page-layout-widget-for-creation.util.ts
msgid "Configuration is required for fields widget"
msgstr ""
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Feb 13, 2026

Choose a reason for hiding this comment

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

P2: New zh-TW translations are added with empty msgstr values, leaving these user-facing errors untranslated. Provide localized strings for the new entries.

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 748:

<comment>New zh-TW translations are added with empty msgstr values, leaving these user-facing errors untranslated. Provide localized strings for the new entries.</comment>

<file context>
@@ -742,6 +742,16 @@ msgstr ""
+#. js-lingui-id: yU57/s
+#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-fields-flat-page-layout-widget-for-creation.util.ts
+msgid "Configuration is required for fields widget"
+msgstr ""
+
+#. js-lingui-id: tj1Qu0
</file context>
Fix with Cubic

#. js-lingui-id: yU57/s
#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-fields-flat-page-layout-widget-for-creation.util.ts
msgid "Configuration is required for fields widget"
msgstr ""
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Feb 13, 2026

Choose a reason for hiding this comment

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

P2: New Japanese locale strings were added without translations (empty msgstr). This leaves the locale incomplete and causes these messages to display in English or as missing translations.

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/ja-JP.po, line 748:

<comment>New Japanese locale strings were added without translations (empty msgstr). This leaves the locale incomplete and causes these messages to display in English or as missing translations.</comment>

<file context>
@@ -742,6 +742,16 @@ msgstr ""
+#. js-lingui-id: yU57/s
+#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-fields-flat-page-layout-widget-for-creation.util.ts
+msgid "Configuration is required for fields widget"
+msgstr ""
+
+#. js-lingui-id: tj1Qu0
</file context>
Fix with Cubic

@@ -742,6 +742,16 @@ msgstr ""
msgid "Composite type not found."
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Feb 13, 2026

Choose a reason for hiding this comment

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

P3: New Korean translations were added with empty msgstr values, which will cause ko-KR users to see English fallback strings. Please provide localized translations for these new messages.

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/ko-KR.po, line 748:

<comment>New Korean translations were added with empty msgstr values, which will cause ko-KR users to see English fallback strings. Please provide localized translations for these new messages.</comment>

<file context>
@@ -742,6 +742,16 @@ msgstr ""
+#. js-lingui-id: yU57/s
+#: src/engine/metadata-modules/flat-page-layout-widget/validators/utils/validate-fields-flat-page-layout-widget-for-creation.util.ts
+msgid "Configuration is required for fields widget"
+msgstr ""
+
+#. js-lingui-id: tj1Qu0
</file context>
Fix with Cubic

@charlesBochet charlesBochet merged commit ebfaff0 into main Feb 13, 2026
17 checks passed
@charlesBochet charlesBochet deleted the i18n branch February 13, 2026 17:19
@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-ff857bb0.json

Generated by 🚫 dangerJS against f0b6186

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