Docs/update documentation - #212
Closed
agualdron wants to merge 8 commits into
Closed
Conversation
Add complete Spanish and Simplified Chinese documentation while preserving English as the default locale. Centralize locale-specific UI and metadata copy, generate localized SEO and sitemap entries, and remove unsupported locale fallbacks. Add *validate:i18n* checks to enforce file parity, protect technical content, and detect untranslated or mixed-language prose.
Contributor
|
Too many files changed for review (526 files, 500 file limit). |
Contributor
|
Important Review skippedToo many files! This PR contains 525 files, which is 425 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (525)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds complete Spanish and Simplified Chinese localization to the TradingGoose documentation while preserving English as the default locale.
This PR also:
en,es, andzh.Why
The documentation previously lacked complete Spanish and Simplified Chinese coverage and relied on translation tooling that did not guarantee exact content parity.
These changes make localized documentation a first-class, validated part of the docs application while keeping technical structures synchronized with English and ensuring newly documented tools, triggers, widgets, and SDK contracts reflect current staging behavior.
Type of Changes
Affected Areas
apps/tradinggooseapps/docspackages/*Validation
Commands and results:
bun run docs:auditPassed with 203/203 documented source features, no missing pages, and no unexpected orphaned pages.
cd apps/docsbun run validate:i18n
Passed with exact parity across 239 English, 239 Spanish, and 239 Chinese documentation files.
cd apps/docsbun run build
Passed. The production Next.js documentation build completed successfully.
bunx biome check --unsafe apps/docs/app apps/docs/components apps/docs/lib apps/docs/scripts scriptsPassed with no branch-owned formatting, lint, or import-order violations.
git grep -n "<<<<<<<"Passed with no conflict markers.
bun run testThe monorepo run completed with 4,511 passing and 33 failing tests. The failures are outside the documentation diff:
listen EPERM.The TypeScript SDK suite passed all 20 tests. Reviewers should confirm the remaining application failures in CI or an unrestricted local environment.
cd apps/docsbun run type-check
The command regenerated the Fumadocs sources but remained active without diagnostics for several minutes and was stopped. Reviewers should confirm the docs type-check in CI.
Reviewer Focus
Please focus on:
validate:i18nguardrails.Risk / Rollout Notes
Config / Data Changes
Screenshots / Video
Not included. Reviewers should spot-check the documentation language dropdown and representative English, Spanish, and Chinese pages in the docs preview.
Checklist