Skip to content

i18n - docs translations#15725

Merged
FelixMalfait merged 1 commit intomainfrom
i18n
Nov 8, 2025
Merged

i18n - docs translations#15725
FelixMalfait merged 1 commit intomainfrom
i18n

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Nov 8, 2025

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.

Greptile Overview

Greptile Summary

This PR fixes internal navigation links in French documentation by adding the /fr/ locale prefix to all internal href paths.

Changes:

  • Updated 63 internal documentation links across 22 French MDX files
  • Fixed both Card component hrefs and markdown link syntax [text](/path) to include /fr/ prefix
  • Ensures proper routing within French localized documentation
  • Maintains consistency with i18n documentation structure

Impact:

  • Resolves broken internal links in French docs that were pointing to English versions
  • Improves French documentation navigation user experience
  • No code logic changes, purely documentation routing fixes

Confidence Score: 5/5

  • This PR is completely safe to merge with zero risk
  • All changes are purely documentation link updates adding /fr/ prefix for proper i18n routing. No code logic, no dependencies, no breaking changes. The pattern is consistent across all 22 files with identical transformation logic.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
packages/twenty-docs/fr/developers/introduction.mdx 5/5 Updated internal links to include /fr/ prefix for French localization
packages/twenty-docs/fr/user-guide/introduction.mdx 5/5 Fixed internal navigation links with /fr/ prefix for proper French docs routing
packages/twenty-docs/fr/user-guide/getting-started/what-is-twenty.mdx 5/5 Corrected internal documentation links to use localized /fr/ paths
packages/twenty-docs/fr/user-guide/workflows/getting-started-workflows.mdx 5/5 Updated workflow documentation links to French localized paths
packages/twenty-docs/fr/user-guide/crm-essentials/contact-and-account-management.mdx 5/5 Fixed CRM documentation internal links with /fr/ prefix

Sequence Diagram

sequenceDiagram
    participant GHA as GitHub Actions
    participant Crowdin
    participant Repo as Repository
    participant Docs as French Docs
    
    GHA->>Crowdin: Fetch translations
    Crowdin->>GHA: Return updated translations
    GHA->>GHA: Fix internal links (add /fr/ prefix)
    GHA->>Repo: Create PR with updated files
    Repo->>Docs: Deploy corrected French docs
    Note over Docs: Users can now navigate<br/>French docs correctly
Loading

22 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@FelixMalfait FelixMalfait merged commit a4ac66c into main Nov 8, 2025
7 of 8 checks passed
@FelixMalfait FelixMalfait deleted the i18n branch November 8, 2025 22:20
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Nov 8, 2025

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:105:5)�[39m
danger-results://tmp/danger-results-46e0095a.json

Generated by 🚫 dangerJS against 6895b4f

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