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

Merge versions of Blazor-SignalR tutorial #30375

Merged
merged 4 commits into from
Sep 15, 2023

Conversation

@guardrex
Copy link
Collaborator Author

@Rick-Anderson @tdykstra @wadepickett ...

[Warning] File aspnetcore/blazor/tutorials/signalr-blazor-preview.md with URL /aspnet/core/blazor/tutorials/signalr-blazor-preview was deleted without redirection. To avoid broken links, add a redirection.

Not every resource placed on the Net should require redirection for removal. This was an extremely point-in-time temporary article that was placed for just a couple of weeks during preview to get me past a workload crunch under high churn. Having to muddy up the redirection file shouldn't be required. I'll add it, but docs management should consider lowering the severity of the warning so that it doesn't prevent merge ..... or perhaps they can simply detect that the file is versioned later than the repo's current version before throwing this warning.

@guardrex guardrex merged commit bc105d8 into main Sep 15, 2023
@guardrex guardrex deleted the guardrex/blazor-merge-signalr-tutorials branch September 15, 2023 13:42
@Rick-Anderson
Copy link
Contributor

Rick-Anderson commented Sep 15, 2023

Not every resource placed on the Net should require redirection for removal.

Agreed. It's a one time warning we can live with one time.

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.

2 participants