-
Notifications
You must be signed in to change notification settings - Fork 116
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
Move all owned NPCs together with player on Team Switching #789
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
worked in a test server and I approve of the code :)
altought this pr won't work if the team is set directly
live footage of this working :
output.mp4
github is mean so I set the bitrate of the video to 100k >:(
Northstar.CustomServers/mod/scripts/vscripts/mp/_gamestate_mp.nut
Outdated
Show resolved
Hide resolved
Northstar.CustomServers/mod/scripts/vscripts/mp/_gamestate_mp.nut
Outdated
Show resolved
Hide resolved
Merging based on previous review. |
I am aware of #721 existence and also address #649, however on doing tests with the PvP glitch in Frontier Defense, the problem of player switching teams does not lie only on Hacked Spectres, the player's own Auto-Titan will not switch, as well whatever Turrets they placed or Ticks they tossed.
This is my approach to fixing all those issue in one go, while also including a missing feature from Respawn's code that notify when players switches teams.