Skip to content

docs: Fix incorrect highlighted line in SignalR documentation#36901

Merged
wadepickett merged 2 commits intodotnet:mainfrom
74dine:main
Mar 20, 2026
Merged

docs: Fix incorrect highlighted line in SignalR documentation#36901
wadepickett merged 2 commits intodotnet:mainfrom
74dine:main

Conversation

@74dine
Copy link
Copy Markdown
Contributor

@74dine 74dine commented Mar 20, 2026

The current SignalR / SignalR with Javascript / Configure SignalR section, highlights an incorrect line.
The highlighted line currently refers to a line generated in a previous step, rather than the new change required to register the hub endpoint.

This PR updates the highlighted line to match the intended code change, aligning it with the SignalR with Blazor tutorial article.


Internal previews

📄 File 🔗 Preview link
aspnetcore/tutorials/signalr.md Customer intent: As a developer, I want to get a quick proof-of-concept app running, so I can get a practical introduction to ASP.NET Core SignalR.

Copy link
Copy Markdown
Contributor

Copilot AI left a 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 corrects the highlighted line in the ASP.NET Core SignalR tutorial so the documentation emphasizes the hub endpoint registration step (MapHub) instead of an unrelated/template-generated line.

Changes:

  • Updates the code snippet highlight range to point to app.MapHub<ChatHub>("/chatHub"); in the Program.cs sample.

@wadepickett
Copy link
Copy Markdown
Contributor

@nis0945, thanks very much for this fix! Greatly appreciated.

Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, approved.

Comment thread aspnetcore/tutorials/signalr.md Outdated
Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

@wadepickett wadepickett merged commit c3b0bb2 into dotnet:main Mar 20, 2026
2 of 4 checks passed
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.

3 participants