Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Multiple User Connections #7188

Merged
merged 20 commits into from
Nov 11, 2022
Merged

Multiple User Connections #7188

merged 20 commits into from
Nov 11, 2022

Conversation

HexaField
Copy link
Member

@HexaField HexaField commented Nov 6, 2022

Summary

  • replace UserId as peer identifier with PeerID
  • refactor avatar spawning to allow multiple connections but only one avatar
  • refactor authority to be peer driven instead of user driven
  • refactor WebRTC channels to use PeerID instead of UserId

References

closes #7069

Checklist

  • If this PR is still a WIP, convert to a draft
  • ensure all checks pass
  • When this PR is ready, mark it as "Ready for review"
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewer

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

@HexaField HexaField changed the title Multiple user connections Multiple User Connections Nov 6, 2022
@HexaField HexaField marked this pull request as ready for review November 10, 2022 10:38
@speigg speigg merged commit 79990ff into dev Nov 11, 2022
@HexaField HexaField deleted the multiple-user-connections branch November 17, 2022 23:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Story]: Support multiple client connections per user
3 participants