Skip to content

Conversation

@Stealthii
Copy link
Contributor

@Stealthii Stealthii commented Apr 8, 2025

Description

The behavior of AUTO_BACKUP_PAUSE_WITH_NO_PLAYERS should be additionally negated when cross play is enabled, as the Steam Query port also doesn't return player counts properly in this situation.

This can be later removed when there is a more concrete method implemented for querying player counts.

Fixes #1050.

Contributions

  • Disallow pausing backups on zero player counts when crossplay is enabled

Checklist

  • I added one or multiple labels which best describes this PR.
  • I have tested the changes locally.
  • This PR has a reviewer on it.
  • I have validated my changes in a docker container and on Ubuntu. (Only needed for Odin or Docker Changes)

Stealthii and others added 5 commits April 8, 2025 13:50
Do to a recent observation in mbround18#1050, the behavior of
AUTO_BACKUP_PAUSE_WITH_NO_PLAYERS should be additionally negated when
cross play is enabled, as the Steam Query port also doesn't return
player counts properly in this situation.

This can be later removed when there is a more concrete method
implemented for querying player counts.
@mbround18
Copy link
Owner

Hey @Stealthii excellent addition! running it thru the checks now :)

@mbround18 mbround18 added docker Tag if its related to docker patch Increment the patch version when merged release Create a release when this pr is merged labels Apr 18, 2025
@mbround18 mbround18 merged commit 2cd9bbb into mbround18:main Apr 18, 2025
2 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Tag if its related to docker patch Increment the patch version when merged release Create a release when this pr is merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Backup issues when crossplay is enabled

2 participants