[v2] added signalr#956
Conversation
|
I've done. We can consider adding Redis for message distribution and scalability in a future iteration. |
|
@wtmaxim could you please resolve the conflicts. |
|
Can this be merged now? |
Need to support redis or azure signalr backplane otherwise can't run in more than one container instance |
I will |
|
Can some usage samples be added for this? We now have the Blazor client ready as well. Atleast from the Products CRUD pov. Can something be added to demonstrate this usage? |
|
@dsolteszopyn @iammukeshm |
|
Hey @wtmaxim , sorry. Forgot about this PR. There is a change in the solution structure now. Whenever you get time, can you resolve these conflicts and let me know. Basically now all the API, Blazor and Aspire Dashboards stay in the same solution file. |
|
|
||
| SignalRSettings? signalRSettings = configuration.GetSection(nameof(SignalRSettings)).Get<SignalRSettings>(); | ||
|
|
||
| if (signalRSettings?.UseBackplane == true) |
There was a problem hiding this comment.
Not the other way round? Backplane should br false here
| } | ||
| else | ||
| { | ||
| var backplaneSettings = configuration.GetSection("SignalRSettings:Backplane").Get<SignalRSettings.Backplane>(); |
There was a problem hiding this comment.
Is there need to read the setting again here? signalRSettings has the settings
| public class Backplane | ||
| { | ||
| public string? Provider { get; set; } | ||
| public string? StringConnection { get; set; } |
|
|
||
| public class SignalRSettings | ||
| { | ||
| public class Backplane |
|
@wtmaxim @iammukeshm what about signalR. Why close this PR?????????? Thanks! |
5 participants
No description provided.