Skip to content

Add demos for chat#868

Draft
xingsy97 wants to merge 7 commits into
mainfrom
s/chat-demos
Draft

Add demos for chat#868
xingsy97 wants to merge 7 commits into
mainfrom
s/chat-demos

Conversation

@xingsy97

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread sdk/webpubsub-chat-client/examples/teams-lite/server/index.js Fixed
@xingsy97
xingsy97 force-pushed the s/chat-demos branch 5 times, most recently from 10ef16f to a8469a3 Compare February 11, 2026 04:00
xingsy97 and others added 5 commits June 30, 2026 12:50
- Replace login() with start() / ChatClient.start()
- Replace addListenerForNewMessage with onMessage (event pattern)
- Replace addListenerForNewRoom with onRoomJoined
- Replace addListenerForMemberJoined/Left with onMemberJoined/Left
- Replace addListenerForRoomLeft with onRoomLeft
- Replace listRoomMessage with listRoomMessages (PagedAsyncIterableIterator)
- Use connection.on() for connection lifecycle events
- Fix HTML sanitization vulnerability in Sidebar.tsx (use DOMParser)
- Fix user input validation in teams-lite server

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Update all package.json to use npm published 1.0.0-beta.2
- Migrate to published event API: client.on('message'|'room-joined'|
  'room-left'|'member-joined'|'member-left'|'started'|'stopped', ...)
- listRoomMessages(roomId) instead of listRoomMessages({ roomId })
- createRoom(title, members, { roomId }) instead of positional 3rd arg
- getRoomDetail(roomId, { withMembers: true }) instead of getRoom()
- ChatClient.start(url) instead of new ChatClient(url) + login/start
- Handle addUserToRoom 'already a member' error in teams-lite server
- Regenerate yarn.lock files for new dependency versions

E2E tested against live Azure Web PubSub resource.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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