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
Replace "Debug" prefix with "Battlefield Control" for gameplay notifications. #12643
Conversation
be405bd
to
20a8ec8
Compare
Added the other messages from #12387, which I had forgotten about. |
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.
Looks otherwise fine, 👍
@@ -58,11 +62,13 @@ public void Tick(Actor self) | |||
|
|||
public void OnPlayerLost(Player player) | |||
{ | |||
Game.Debug("{0} is defeated.", player.PlayerName); | |||
if (info.SuppressNotifications) | |||
return; |
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.
You'll probably want to move that to below the foreach
, otherwise it would be suppressing more than just the notifications.
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.
Could have sworn that i'd done that... anyway, fixed.
20a8ec8
to
e1f7e5d
Compare
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.
LGTM. 👍
This also introduces a
SuppressNotifications
flag that maps can use to disable the default messages if they want to display something different using Lua.