doc(Tutorials): remove Tutorials route#8046
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideRemoves the obsolete top-level Tutorials page and route, updates the navigation header to point directly to the tutorials dashboard sub-route, and cleans up associated localized resources and styles. File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Since the
/tutorialsroute is being removed, consider adding a redirect or route alias from the old URL to/tutorials/dashboardso existing bookmarks and external links don’t break. - Double-check any remaining references to the old
Tutorialspage (e.g., routes, menus, or components) to ensure there are no dead links or unused localization entries after deletingTutorials.razorand its styles.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Since the `/tutorials` route is being removed, consider adding a redirect or route alias from the old URL to `/tutorials/dashboard` so existing bookmarks and external links don’t break.
- Double-check any remaining references to the old `Tutorials` page (e.g., routes, menus, or components) to ensure there are no dead links or unused localization entries after deleting `Tutorials.razor` and its styles.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Pull request overview
Removes the obsolete /tutorials landing route from the Server website and updates navigation/localization to point users to the Tutorials section’s dashboard entry point.
Changes:
- Deleted the
/tutorialspage (and its scoped CSS) to remove the Tutorials landing route. - Updated the top header navigation to link to
tutorials/dashboardinstead oftutorials. - Cleaned up related localization entries (removed unused
CategoryTitle; minor JitViewer string tweaks).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/BootstrapBlazor.Server/Locales/zh-CN.json | Removes the unused CategoryTitle string under TutorialsNavMenu. |
| src/BootstrapBlazor.Server/Locales/en-US.json | Removes CategoryTitle under TutorialsNavMenu; adjusts JitViewer-related display strings. |
| src/BootstrapBlazor.Server/Components/Pages/Tutorials.razor.css | Deletes scoped CSS for the removed Tutorials landing page. |
| src/BootstrapBlazor.Server/Components/Pages/Tutorials.razor | Deletes the /tutorials routed page. |
| src/BootstrapBlazor.Server/Components/Components/Header.razor | Updates the header Tutorials link to tutorials/dashboard (and normalizes the first-line BOM/formatting). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8046 +/- ##
========================================
Coverage ? 100.00%
========================================
Files ? 766
Lines ? 34157
Branches ? 4697
========================================
Hits ? 34157
Misses ? 0
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Link issues
fixes #8045
Summary By Copilot
Regression?
Risk
Verification
Packaging changes reviewed?
☑️ Self Check before Merge
Summary by Sourcery
Update navigation to point Tutorials menu to the dashboard and remove the legacy Tutorials page.
Enhancements:
Chores: