Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/i18n #67

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Bardreamaster
Copy link
Contributor

@Bardreamaster Bardreamaster commented Apr 25, 2024

Type of PR

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • πŸ§™β€β™‚οΈ New feature (adding a feature following a feature-request issue)
  • πŸ”¨ Improvement (improving an existing feature - includes style: and perf:commits)
  • πŸ—οΈ Refactor (rewriting existing code without any feature change)
  • ✍️ (!) This change is or requires a documentation update

Description

Add Chinese documents for all version 2 docs.

Fix some format/spell error in current docs.

Changes

Contextual Changes:

  • provide bullet list of overview to each context.

Additional Screenshots

Provide screenshot if necessary.

Additional information/context

If relevant, add any information or context that would be useful to evaluate this PR

To checkout the preview locally, run this command in /documentation folder: npm run start -- --locale zh

then you can fell free check the new docs.

While, there are still a problem needs your help:

there's no valid route for default page in http://127.0.0.1:3000/zh/ if you visit this URL directly.

but if you click the button on top ζ–‡ζ‘£, contents would be available with the same URL.

And after fix the problem above, I may add some guides for i18n if someone want to add other language support.

It has been quiet a long time, hope for your reply.

@Bardreamaster
Copy link
Contributor Author

@Kinostrome if you have time to check my PR, thanks so much

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