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

Update readme for Azure SignalR hosting #3410

Merged
merged 1 commit into from
Apr 7, 2024
Merged

Update readme for Azure SignalR hosting #3410

merged 1 commit into from
Apr 7, 2024

Conversation

timheuer
Copy link
Member

@timheuer timheuer commented Apr 4, 2024

Fixes #3390 by updating readme to note that the named SignalR method on the client will leverage the created connection string, matching our documentation already.

Microsoft Reviewers: Open in CodeFlow

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Apr 4, 2024
@davidfowl
Copy link
Member

cc @vicancy

@davidfowl davidfowl added this to the 8.1 milestone Apr 6, 2024
@davidfowl davidfowl merged commit 0e18cbc into main Apr 7, 2024
8 checks passed
@davidfowl davidfowl deleted the timheuer/3390 branch April 7, 2024 03:32
@davidfowl
Copy link
Member

/backport to release/8.0

Copy link
Contributor

github-actions bot commented Apr 7, 2024

Started backporting to release/8.0: https://github.com/dotnet/aspire/actions/runs/8585908367

Copy link
Contributor

@vicancy vicancy left a comment

Choose a reason for hiding this comment

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

One minor comment.

@@ -27,9 +27,18 @@ var myService = builder.AddProject<Projects.MyService>()
.WithReference(signalR);
```

The `WithReference` method configures a connection in the `MyService` project named `sr`. In the _Program.cs_ file of `MyService`, the Azure SignalR connection can be consumed using the client library [Microsoft.Azure.SignalR](https://www.nuget.org/packages/Microsoft.Azure.SignalR):
Copy link
Contributor

Choose a reason for hiding this comment

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

Azure SignalR SDK is not a client library.

Suggested change
The `WithReference` method configures a connection in the `MyService` project named `sr`. In the _Program.cs_ file of `MyService`, the Azure SignalR connection can be consumed using the client library [Microsoft.Azure.SignalR](https://www.nuget.org/packages/Microsoft.Azure.SignalR):
The `WithReference` method configures a connection in the `MyService` project named `sr`. In the _Program.cs_ file of `MyService`, the Azure SignalR connection can be consumed using the library [Microsoft.Azure.SignalR](https://www.nuget.org/packages/Microsoft.Azure.SignalR):

@github-actions github-actions bot locked and limited conversation to collaborators May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Azure SignalR hosting readme to note ConnectionString usage and client methods to use
3 participants