Skip to content

i18n - docs translations#16185

Merged
FelixMalfait merged 2 commits intomainfrom
i18n
Nov 28, 2025
Merged

i18n - docs translations#16185
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 Nov 28, 2025

Greptile Overview

Greptile Summary

This PR updates documentation translations and cleans up .po locale file metadata from Crowdin. The .po file changes are safe (removing duplicate source references), but the documentation MDX files have critical issues:

  • Japanese documentation files contain severe text duplication where phrases are repeated 10-30 times consecutively
  • Chinese documentation file has nested warning tags and repeated content blocks
  • The PR changes make these issues worse by adding more duplicate lines instead of fixing them

The .po file updates (31 files) are clean and simply remove duplicate source file reference comments. However, the documentation translation files appear to have corruption from the translation pipeline that needs to be addressed before merging.

Confidence Score: 1/5

  • This PR has critical documentation corruption issues that make it unsafe to merge
  • Score reflects severe text duplication bugs in Japanese and Chinese documentation files where phrases are repeated 10-30 times consecutively, making content unreadable. The changes worsen existing problems by adding more duplicates. While .po files are clean, the documentation corruption must be fixed before merging.
  • Critical attention needed: packages/twenty-docs/l/ja/developers/frontend-development/work-with-figma.mdx, packages/twenty-docs/l/ja/user-guide/data-model/customize-your-data-model.mdx, and packages/twenty-docs/l/zh/user-guide/settings/profile-settings.mdx - all have severe content duplication issues

Important Files Changed

File Analysis

Filename Score Overview
packages/twenty-docs/l/ja/developers/frontend-development/work-with-figma.mdx 2/5 Added duplicate line - appears to be unintended repetition in Japanese translation
packages/twenty-docs/l/ja/user-guide/data-model/customize-your-data-model.mdx 2/5 Added one more duplicate phrase - exacerbates existing repetition issue
packages/twenty-docs/l/zh/user-guide/settings/profile-settings.mdx 2/5 Added whitespace and removed blank line - file still has severe duplication issues
packages/twenty-front/src/locales/en.po 5/5 Removed duplicate source file references - cleanup of .po metadata

Sequence Diagram

sequenceDiagram
    participant GH as GitHub Actions
    participant Crowdin
    participant Repo as Repository
    participant Docs as Documentation MDX
    participant PO as .po Locale Files
    
    Note over GH,Crowdin: Translation Update Workflow
    
    Crowdin->>GH: Trigger translation sync
    GH->>Repo: Pull latest translations
    
    par Documentation Updates
        GH->>Docs: Update Japanese translations
        Note over Docs: work-with-figma.mdx<br/>docker-compose.mdx<br/>customize-your-data-model.mdx
        GH->>Docs: Update Chinese translations
        Note over Docs: profile-settings.mdx
    and Locale File Updates
        GH->>PO: Update 31 .po files
        Note over PO: Remove duplicate source<br/>file references
    end
    
    GH->>Repo: Create PR #16185
    Note over Repo: Automated i18n update
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.

Additional Comments (1)

  1. packages/twenty-docs/l/zh/user-guide/settings/profile-settings.mdx, line 40-50 (link)

    logic: Severe duplication issue with multiple copies of the same warning text in both simplified and traditional Chinese. The file has nested </Warning> tags and repeated content that makes the documentation unreadable. The change added a space but doesn't address the fundamental duplication problem.

35 files reviewed, 4 comments

Edit Code Review Agent Settings | Greptile

Comment on lines 107 to +110
これは推奨拡張機能の一部です。
これは推奨拡張機能の一部です。
これは推奨拡張機能の一部です。
これは推奨拡張機能の一部です。
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

logic: Duplicate line added makes existing repetition worse. This file already has 10+ identical lines saying "これは推奨拡張機能の一部です。" (This is part of recommended extensions). Adding one more line increases the problem rather than fixing it.

Prompt To Fix With AI
This is a comment left during a code review.
Path: packages/twenty-docs/l/ja/developers/frontend-development/work-with-figma.mdx
Line: 107:110

Comment:
**logic:** Duplicate line added makes existing repetition worse. This file already has 10+ identical lines saying "これは推奨拡張機能の一部です。" (This is part of recommended extensions). Adding one more line increases the problem rather than fixing it.

How can I resolve this? If you propose a fix, please make it concise.

@@ -30,7 +30,7 @@ Twentyは、日常をサポートする最適なデータモデルを形成す
既存のオブジェクトの特性に過ぎないもの(例:会社の「業種」や機会の「ステータス」)はフィールドにします。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。 カテゴリ、ラベル、属性にはフィールドが最適です。
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

logic: Line contains 23 repetitions of "カテゴリ、ラベル、属性にはフィールドが最適です。" (Fields are best for categories, labels, and attributes). Should be a single occurrence.

Prompt To Fix With AI
This is a comment left during a code review.
Path: packages/twenty-docs/l/ja/user-guide/data-model/customize-your-data-model.mdx
Line: 30:30

Comment:
**logic:** Line contains 23 repetitions of "カテゴリ、ラベル、属性にはフィールドが最適です。" (Fields are best for categories, labels, and attributes). Should be a single occurrence.

How can I resolve this? If you propose a fix, please make it concise.


**3. 単独で存在する場合には新しいオブジェクトを作成してください。**
概念が独自のライフサイクル、プロパティ、または関係を持つ場合、それは通常オブジェクトに値します。 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば:
概念が独自のライフサイクル、プロパティ、または関係を持つ場合、それは通常オブジェクトに値します。 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば: 例えば:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

logic: Line contains 30 repetitions of "例えば:" (For example:). The change added one more repetition, making it 30 instead of 29. Should be a single occurrence.

Prompt To Fix With AI
This is a comment left during a code review.
Path: packages/twenty-docs/l/ja/user-guide/data-model/customize-your-data-model.mdx
Line: 33:33

Comment:
**logic:** Line contains 30 repetitions of "例えば:" (For example:). The change added one more repetition, making it 30 instead of 29. Should be a single occurrence.

How can I resolve this? If you propose a fix, please make it concise.

@FelixMalfait FelixMalfait merged commit df20c52 into main Nov 28, 2025
9 of 10 checks passed
@FelixMalfait FelixMalfait deleted the i18n branch November 28, 2025 16:21
@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:6433:19)�[39m
�[90m    at successSteps (node:internal/deps/undici/undici:6414:27)�[39m
�[90m    at readAllBytes (node:internal/deps/undici/undici:5380:13)�[39m
�[90m    at process.processTicksAndRejections (node:internal/process/task_queues:103:5)�[39m
danger-results://tmp/danger-results-0ef531ac.json

Generated by 🚫 dangerJS against 8dab9ef

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