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

Do not disband alt bots group when master goes away #702

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EricksOliveira
Copy link
Contributor

No description provided.

@hermensbas
Copy link
Collaborator

@noisiver whats your opinion on this? Shouldnt alt-bots group disband after the master leaves?

@noisiver
Copy link
Collaborator

It's hard to say. If there are other players, should one of them become the new master or not? It depends a lot on the players and doesn't really have a one-size-fits-all solution. If there are no other players it makes no sense not to disband the group.

@noisiver
Copy link
Collaborator

It shouldn't disband the group if the master actually is in the group though. I mean.. I log out with altbots in my party/raid all the time, I wouldn't want it to disband the group just because I go offline when they do. They won't log in again without me so as long as you're actually online (and in the group) when they are it shouldn't disband the group.

@arcaneintellect
Copy link

I would love something like this implemented, bonus if its behind a config or something for those who want it to auto disband. But at this time I cant come up with a viable strategy for swapping altbots in the same dungeon/raid without them teleporting to the graveyard.

@nl-saw
Copy link
Collaborator

nl-saw commented Jan 20, 2025

@EricksOliveira, can you perhaps cover this as in a config option?

@nl-saw nl-saw requested review from liyunfan1223 and removed request for liyunfan1223 January 23, 2025 10:58
@nl-saw nl-saw self-assigned this Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants