-
Notifications
You must be signed in to change notification settings - Fork 25.3k
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
SignalR Client Features doc -> SignalR Clients overview doc #18293
Conversation
TOC update preview: |
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.
approved provided Addie's suggestions go in. And @halter73's suggestion in the thread would make me very happy and I think mitigate lots of confusion with customers. If it would also simplify the engineering process, let's discuss how to make it happen.
Co-authored-by: Scott Addie <[email protected]>
@anurse Looks good. Merge when you're ready. Consider that contraction a parting gift for you. 😄 |
and while we're at it, I have to get one last lonely man usage in there. |
😭😭😭😭😭😭😭 |
In reviewing dotnet/aspnetcore#21618 and other issues about compatibility and versioning I figured this doc could use a quick update. Here's a first stab at it. I'm going to check the TOCs shortly and look at the internal review site, since I changed some titles.