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

Adds an auto balancing system for the NCR and Legion (TM ONLY) #180

Closed
wants to merge 2 commits into from

Conversation

xPokee
Copy link
Contributor

@xPokee xPokee commented May 25, 2024

About The Pull Request

If either NCR or Legion (or both) have more than four players, the system will check if NCR/Legion has double the players of the other faction, if they do it disables the spawns for the NCR/Legion until they get more players. It has both a config toggle and an admin button toggle.

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

Changelog

🆑
add: added a faction auto balancing system
config: added config lines for enabling/disabling the auto balancing by default
admin: gave admins a button to toggle off/on the auto balancing (good for events)
/:cl:

@xTrainx xTrainx added test merged PR is currently test merged hot topic This PR is under debate needs work Something isn't right with this PR faction changes Changes something involving factions. balance This PR messes with the general balance of the game labels May 26, 2024
@xPokee
Copy link
Contributor Author

xPokee commented Jun 16, 2024

This has never worked to my knowledge and with our current player count I can no longer justify having this addition. Maybe I'll open the PR back up if I either fix it or if we get more players.

@xPokee xPokee closed this Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
balance This PR messes with the general balance of the game faction changes Changes something involving factions. hot topic This PR is under debate needs work Something isn't right with this PR test merged PR is currently test merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants