Closed xingsy97 closed 6 months ago
@manekinekko what's the current status? Can we proceed to merge without refactoring changes?
I was working on adding bicep/azd iac part of this PR, but I am hitting a blocking issue. I am going to merge this PR as-is and will add bicep/azd deployment in a different PR. Will also refactor code in a separate PR.
I was working on adding bicep/azd iac part of this PR, but I am hitting a blocking issue. I am going to merge this PR as-is and will add bicep/azd deployment in a different PR. Will also refactor code in a separate PR.
Agreed. Can we merge before the event coming up?
Summary
add realtime notification feature for portal powered by Azure Web PubSub For Socket.IO
This draft PR doesn't contain azure deployment code. So user have to use a pre-created Web PubSub resource, manually replace endpoint in
packages/portal
and add connection string inpackages/realtime
How to use
key
tab in resource portal, copy connection string..env.example
underpackages/realtime/
to.env
. Then fill in copied connection string to variableWebPubSubConnectionString
https://<resource-name>.webpubsub.azure.com
). And use it replace the default endpoint inpackages/portal/src/app/shared/realtime.service.ts:L14
Feature specification
The realtime notification has four scenarios as cited below. See complete specification in the internal doc.
Other reference
https://learn.microsoft.com/en-us/azure/azure-web-pubsub/socketio-migrate-from-self-hosted