Skip to content

i18n - docs translations#17149

Merged
charlesBochet merged 1 commit intomainfrom
i18n-docs
Jan 14, 2026
Merged

i18n - docs translations#17149
charlesBochet merged 1 commit intomainfrom
i18n-docs

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Created by Github action

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.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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.

2 issues found across 10 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-docs/l/ar/twenty-ui/navigation/step-bar.mdx">

<violation number="1" location="packages/twenty-docs/l/ar/twenty-ui/navigation/step-bar.mdx:32">
P2: Keep API identifiers untranslated. The StepBar component still expects the `activeStep` prop, so the table should list that exact name instead of translating it to Arabic.</violation>
</file>

<file name="packages/twenty-docs/l/ar/twenty-ui/input/radio.mdx">

<violation number="1" location="packages/twenty-docs/l/ar/twenty-ui/input/radio.mdx:40">
P2: The Radio usage example now includes an extraneous `},{` line, resulting in invalid JavaScript in the documentation snippet.</violation>
</file>

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

| activeStep | رقم | مؤشر للخطوة النشطة حاليًا. هذا يحدد أي خطوة يجب إبرازها بشكل مرئي |
| المحددات | النوع | الوصف |
| ------------- | ----- | ----------------------------------------------------------------- |
| الخطوة النشطة | رقم | مؤشر للخطوة النشطة حاليًا. هذا يحدد أي خطوة يجب إبرازها بشكل مرئي |
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.

P2: Keep API identifiers untranslated. The StepBar component still expects the activeStep prop, so the table should list that exact name instead of translating it to Arabic.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-docs/l/ar/twenty-ui/navigation/step-bar.mdx, line 32:

<comment>Keep API identifiers untranslated. The StepBar component still expects the `activeStep` prop, so the table should list that exact name instead of translating it to Arabic.</comment>

<file context>
@@ -27,8 +27,8 @@ image: /images/user-guide/api/api.png
-    | activeStep | رقم   | مؤشر للخطوة النشطة حاليًا. هذا يحدد أي خطوة يجب إبرازها بشكل مرئي |
+    | المحددات      | النوع | الوصف                                                             |
+    | ------------- | ----- | ----------------------------------------------------------------- |
+    | الخطوة النشطة | رقم   | مؤشر للخطوة النشطة حاليًا. هذا يحدد أي خطوة يجب إبرازها بشكل مرئي |
   </Tab>
 </Tabs>
</file context>
Suggested change
| الخطوة النشطة | رقم | مؤشر للخطوة النشطة حاليًا. هذا يحدد أي خطوة يجب إبرازها بشكل مرئي |
| activeStep | رقم | مؤشر للخطوة النشطة حاليًا. هذا يحدد أي خطوة يجب إبرازها بشكل مرئي |

);
};

},{
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.

P2: The Radio usage example now includes an extraneous },{ line, resulting in invalid JavaScript in the documentation snippet.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/twenty-docs/l/ar/twenty-ui/input/radio.mdx, line 40:

<comment>The Radio usage example now includes an extraneous `},{` line, resulting in invalid JavaScript in the documentation snippet.</comment>

<file context>
@@ -37,22 +37,22 @@ image: /images/user-guide/create-workspace/workspace-cover.png
       );
     };
-
+    },{
     ```
   </Tab>
</file context>
Suggested change
},{

@charlesBochet charlesBochet merged commit 3ecfb24 into main Jan 14, 2026
12 of 13 checks passed
@charlesBochet charlesBochet deleted the i18n-docs branch January 14, 2026 12:46
@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-57ce8ceb.json

Generated by 🚫 dangerJS against 973f914

lucasbordeau pushed a commit to vasu1303/twenty that referenced this pull request Jan 21, 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