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

allow autohosts to open multiple battles #58

Open
abma opened this issue Dec 29, 2013 · 7 comments
Open

allow autohosts to open multiple battles #58

abma opened this issue Dec 29, 2013 · 7 comments

Comments

@abma
Copy link
Contributor

abma commented Dec 29, 2013

it should be possible for bots to open multiple battle at once.

migration path:

  • handle battles similar to channels
  • distingish battle / channel by channel name, numbers only = battle, name with chars = channel
  • in battle, channel owner = host
  • battle update commands needs channel name as param

commands needed to be changed:

REMOVESCRIPTTAGS
SETSCRIPTTAGS
SCRIPTSTART
SCRIPTEND
ADDSTARTRECT
REMOVESTARTRECT
UPDATEBOT
REMOVEBOT
ENABLEUNITS
ENABLEALLUNITS
DISABLEUNITS
FORCESPECTATORMODE
FORCEALLYNO
FORCETEAMNO
KICKFROMBATTLE
HANDICAP
MYBATTLESTATUS
LEAVEBATTLE

...

(ugh, list is larger than i thought)

@abma
Copy link
Contributor Author

abma commented Dec 29, 2013

related to #57

@abma
Copy link
Contributor Author

abma commented Jan 17, 2015

can be easily done after #135 is implemented

@silentwings
Copy link
Contributor

n2s, since channels save stuff to db, can't use battle_id for channel name

@abma
Copy link
Contributor Author

abma commented Oct 7, 2018

add the attribute battle_id to channels? :-P

@silentwings
Copy link
Contributor

I'm working on making Battle a derived class of Channel!

@silentwings
Copy link
Contributor

silentwings commented Oct 8, 2018

n2s ideas:

FORCETEAMNO, FORCEALLYNO, FORCETEAMCOLOR, FORCESPECTATORMODE, HANDICAP
should be deprecated/combined into a single key=value command e.g.
SETUSERBATTLESTATUS

this leaves the following commands needing changes:
JOINBATTLE LEAVEBATTLE KICKFROMBATTLE
SETSCRIPTTAGS REMOVESCRIPTTAGS
ADDBOT UPDATEBOT REMOVEBOT
RING
plus the following which should probably be subsumed into *SCRIPTTAGS
ADDSTARTRECT REMOVESTARTRECT
ENABLEUNITS ENABLEALLUNITS DISABLEUNITS

@silentwings
Copy link
Contributor

a1c0e85

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants