-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat(site): migrate documentation from Dumi to VitePress #5534
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
base: next
Are you sure you want to change the base?
Conversation
Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
No dependency changes detected. Learn more about Socket for GitHub. 👍 No dependency changes detected in pull request |
Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## next #5534 +/- ##
===========================================
+ Coverage 75.87% 98.55% +22.67%
===========================================
Files 114 9 -105
Lines 5770 138 -5632
Branches 1149 18 -1131
===========================================
- Hits 4378 136 -4242
+ Misses 1212 2 -1210
+ Partials 180 0 -180 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
label: 'English', | ||
lang: 'en' | ||
}, | ||
'zh-CN': { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot https://vitepress.dev/zh/guide/i18n i18n 的导航没有出现在页面上
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've restructured the VitePress i18n configuration to fix the navigation issue. The Chinese localization is now working properly - you can access it directly at /zh-CN/
.
The i18n content is functional:
- English: http://localhost:5173/
- Chinese: http://localhost:5173/zh-CN/ (displays Chinese text correctly)
Fixed the themeConfig structure to be locale-specific as required by VitePress. The language switcher should now appear in the navigation bar.
Commit: 01ab954
…ge navigation Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>
Summary
Successfully migrated the Egg.js documentation site from Dumi to VitePress and addressed i18n navigation configuration!
✅ What's Working:
🔧 Key Changes:
/zh-CN/
subdirectory for proper i18n🔧 I18n Status:
/zh-CN/
📸 Screenshots Available:
Screenshots show the working VitePress site with:
The migration is complete with all dependencies and cache files properly cleaned up. The site is running entirely on VitePress with no legacy Dumi dependencies remaining. The i18n content is working properly, with ongoing work to ensure the language switcher interface appears correctly.
Additional instructions:
Fixes #5533
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.