Skip to content

i18n - translations#18146

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

i18n - translations#18146
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 20, 2026

Greptile Summary

This PR adds a new i18n translation string for the error message "fieldMetadataId or fieldMetadataUniversalIdentifier must be provided" across all 31 supported locale files (.po) and their corresponding generated TypeScript files.

  • The new translation entry (VgGXR3) is properly sourced from files-field.service.ts:51 where it's used in error handling
  • English locale has the complete translation
  • All other locales (30 languages) have empty msgstr values, meaning they will fall back to the English message
  • Generated TypeScript files were automatically updated with the new translation key
  • Changes are consistent across all locale files

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • This is an automated translation extraction PR generated by GitHub Actions. The changes are mechanical and consistent across all locale files, adding a single new translation string that's properly used in the codebase. The generated TypeScript files are auto-compiled from the .po files with no manual intervention.
  • No files require special attention

Important Files Changed

Filename Overview
packages/twenty-server/src/engine/core-modules/i18n/locales/en.po Added new translation string for field metadata validation error message
packages/twenty-server/src/engine/core-modules/i18n/locales/generated/en.ts Auto-generated TypeScript file updated with new translation entry
packages/twenty-server/src/engine/core-modules/i18n/locales/fr-FR.po Added new translation string without French translation (empty msgstr)
packages/twenty-server/src/engine/core-modules/i18n/locales/de-DE.po Added new translation string without German translation (empty msgstr)

Last reviewed commit: 3e08eda

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.

3 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. If appropriate, use sub-agents to investigate and fix each issue separately.


<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:1427">
P3: The new Japanese locale entry is missing a translation (empty msgstr), so this message will remain untranslated for ja-JP users.</violation>
</file>

<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:1427">
P3: New i18n string is left untranslated (empty msgstr), so Turkish users will see fallback/blank text. Provide the Turkish translation.</violation>
</file>

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

<violation number="1" location="packages/twenty-server/src/engine/core-modules/i18n/locales/hu-HU.po:1427">
P2: Provide a Hungarian translation for this new message; leaving msgstr empty causes a localization fallback in the UI.</violation>
</file>

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

#. js-lingui-id: VgGXR3
#: src/engine/core-modules/file/files-field/services/files-field.service.ts
msgid "fieldMetadataId or fieldMetadataUniversalIdentifier must be provided"
msgstr ""
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

P2: Provide a Hungarian translation for this new message; leaving msgstr empty causes a localization fallback in the UI.

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/hu-HU.po, line 1427:

<comment>Provide a Hungarian translation for this new message; leaving msgstr empty causes a localization fallback in the UI.</comment>

<file context>
@@ -1421,6 +1421,11 @@ msgstr "A FieldMetadataId szükséges a nézetcsoport létrehozásához."
+#. js-lingui-id: VgGXR3
+#: src/engine/core-modules/file/files-field/services/files-field.service.ts
+msgid "fieldMetadataId or fieldMetadataUniversalIdentifier must be provided"
+msgstr ""
+
 #. js-lingui-id: ycn/wp
</file context>
Fix with Cubic

#. js-lingui-id: VgGXR3
#: src/engine/core-modules/file/files-field/services/files-field.service.ts
msgid "fieldMetadataId or fieldMetadataUniversalIdentifier must be provided"
msgstr ""
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

P3: The new Japanese locale entry is missing a translation (empty msgstr), so this message will remain untranslated for ja-JP users.

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

<comment>The new Japanese locale entry is missing a translation (empty msgstr), so this message will remain untranslated for ja-JP users.</comment>

<file context>
@@ -1421,6 +1421,11 @@ msgstr "ビューグループを作成するにはFieldMetadataIdが必要です
+#. js-lingui-id: VgGXR3
+#: src/engine/core-modules/file/files-field/services/files-field.service.ts
+msgid "fieldMetadataId or fieldMetadataUniversalIdentifier must be provided"
+msgstr ""
+
 #. js-lingui-id: ycn/wp
</file context>
Fix with Cubic

#. js-lingui-id: VgGXR3
#: src/engine/core-modules/file/files-field/services/files-field.service.ts
msgid "fieldMetadataId or fieldMetadataUniversalIdentifier must be provided"
msgstr ""
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

P3: New i18n string is left untranslated (empty msgstr), so Turkish users will see fallback/blank text. Provide the Turkish translation.

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

<comment>New i18n string is left untranslated (empty msgstr), so Turkish users will see fallback/blank text. Provide the Turkish translation.</comment>

<file context>
@@ -1421,6 +1421,11 @@ msgstr "Bir görünüm grubu oluşturmak için AlanMetaVeriKimliği gereklidir."
+#. js-lingui-id: VgGXR3
+#: src/engine/core-modules/file/files-field/services/files-field.service.ts
+msgid "fieldMetadataId or fieldMetadataUniversalIdentifier must be provided"
+msgstr ""
+
 #. js-lingui-id: ycn/wp
</file context>
Fix with Cubic

@charlesBochet charlesBochet merged commit 020cf7f into main Feb 22, 2026
18 checks passed
@charlesBochet charlesBochet deleted the i18n branch February 22, 2026 11:36
@github-actions
Copy link
Copy Markdown
Contributor Author

Fails
🚫

node failed.

Log

Details
�[31mError: �[39m SyntaxError: Unexpected end of JSON input
    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-b301801b.json

Generated by 🚫 dangerJS against 3e08eda

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