-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Get started tutorial updates #35786
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
Get started tutorial updates #35786
Conversation
Co-authored-by: Copilot <[email protected]>
…tnet/AspNetCore.Docs into guardrex/get-started-tutorial
danroth27
left a comment
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.
The changes to Blazor look good. You found a couple of mentions of Razor Pages that we'll want to clean up too, but it doesn't have to be in this PR.
|
@danroth27 ... 👍 ... I'll open issues for anything that I can't fix quickly-ish. Images will go in tomorrow. I'll ping everyone for review. |
|
btw @tdykstra @wadepickett ... WRT the build report remark that ...
Do we have a replacement? If so, I'll just stick it on this PR for that whole block of articles to silence this from popping up in the future. ... and that YARP article node is ... yikes! It needs work. I pinged on the YARP organizational issue about it. I can't fix those articles here. |
danroth27
left a comment
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.
![]()
wadepickett
left a comment
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.
@guardrex, looks great! I am not spotting any issues at all. Well done.
tdykstra
left a comment
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.
lookss great, just a few nit suggestions
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.
Pull Request Overview
This PR updates the ASP.NET Core "Get started" tutorial by modernizing the content and reorganizing files. The changes replace the old Razor Pages-focused tutorial with a new Blazor Web App tutorial for ASP.NET Core 8.0+, while maintaining backward compatibility for earlier versions.
- Consolidates the getting started content from a folder structure to a single file (
get-started.md) - Updates the tutorial to focus on Blazor Web Apps for modern ASP.NET Core versions
- Fixes broken image references by updating paths from
~/getting-started/_static/to~/static/
Reviewed Changes
Copilot reviewed 22 out of 40 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| aspnetcore/get-started.md | New consolidated tutorial file with Blazor Web App focus for 8.0+ and Razor Pages for earlier versions |
| aspnetcore/getting-started/index.md | Removes old tutorial file (replaced by get-started.md) |
| aspnetcore/getting-started/sample/Index.cshtml | Removes sample file no longer needed |
| aspnetcore/whats-new/dotnet-AspNetCore.Docs-mod0.md | Updates cross-reference from getting-started to get-started |
| Multiple include files | Updates image paths from ~/getting-started/_static/ to ~/static/ |
| .openpublishing.redirection.json | Adds redirect from old getting-started path to new get-started path |
Fixes #35740
Notes
cd) command from the command that runs the app. Shells behave differently when multiple commands are pasted at once. My approach avoids 😈._staticfiles here that are ancient and defunct. Removing them here.getting-startedfolder for some reason. Moving them here.getting-startedfolder. Moving it here.Internal previews