[GH-1831] Add toggle bots button

[Nico-Sanchez]

[!IMPORTANT]
This PR must be merged along with https://github.com/lambdaclass/mirra_backend/pull/693

Closes #1831

Motivation

We need a way to disable/enable the bots as pleased, for testing purposes.

Summary of changes

• Add new ToggleBots protobuf message
• Add new ToggleBots button in battle settings controller
• Add new ToggleBots button action in battle script and its websocket message
• Add ToggleBots button to BattleSettingsSplash prefab

How has this been tested?

Just play a game and toggle BOTS in settings > dev tab. Make sure you use the following backend branch -> https://github.com/lambdaclass/mirra_backend/pull/693

Checklist

• [x] I have tested the changes locally.
• [x] I have tested the whole game after applying the changes, not only the affected areas.
• [x] I self-reviewed the changes on GitHub, line by line.
• [x] I have tested the changes in another devices.
  • [ ] Tested in iOS.
  • [x] Tested in Android.
• [ ] This change requires new documentation.
  • [ ] Documentation has been added/updated.

[Nico-Sanchez]

Do you think something could be made to avoid this message? Or is it printed for a reason?

This happens when bots are turned on or off, since the toggle bots message that the backend broadcasts isn't handled in GameServerConnectionManager.cs

I didn't add this print and I find it useful. You can see in your console any unhandled message you receive.

[tkz00]

Do you think something could be made to avoid this message? Or is it printed for a reason? This happens when bots are turned on or off, since the toggle bots message that the backend broadcasts isn't handled in GameServerConnectionManager.cs

I didn't add this print and I find it useful. You can see in your console any unhandled message you receive.

Yeah, I meant to disable the print specifically for the message ToggleBots to not clog the console, since it's normal the client doesn't handle it, I believe the correct way of "handling" that message is doing nothing, and leaving the log for any other unexpected messages, but if you feel it's better to leave it for visibility it's ok too.

[Nico-Sanchez]

Do you think something could be made to avoid this message? Or is it printed for a reason? This happens when bots are turned on or off, since the toggle bots message that the backend broadcasts isn't handled in GameServerConnectionManager.cs

I didn't add this print and I find it useful. You can see in your console any unhandled message you receive.

Yeah, I meant to disable the print specifically for the message ToggleBots to not clog the console, since it's normal the client doesn't handle it, I believe the correct way of "handling" that message is doing nothing, and leaving the log for any other unexpected messages, but if you feel it's better to leave it for visibility it's ok too.

Oh I get what you mean, you're totally right. I'll fix it.