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

feat(networking): Add 'DisconnectedPlayers' resource to net session runner #409

Merged
merged 1 commit into from
Jun 8, 2024

Conversation

MaxCWhitehead
Copy link
Collaborator

Each frame DisconnectedPlayers resource is updated in net session, can be used by game to determine which players may not be connected.

If rollback before a disconnect - player is still listed in resource, because dc'd players are persisted on runner directly + copied to resource.

@MaxCWhitehead MaxCWhitehead force-pushed the disconnected-players branch from e63939a to 4632e0c Compare June 8, 2024 05:47
@MaxCWhitehead MaxCWhitehead merged commit 8a9492b into fishfolk:main Jun 8, 2024
10 checks passed
github-merge-queue bot pushed a commit to fishfolk/jumpy that referenced this pull request Jun 8, 2024
During a game, if all other players have been disconnected, UI pop-up
will notify player providing button to return to main menu.

If one player but not all have disconnected, going to have game keep
going / figure out how to handle this separately.

Relates to #994, will not fix
anything, but if hung due to disconnect, will inform player at least.

Depends on fishfolk/bones#409, CI will fail
until merged + update bones version.
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.

1 participant