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

Feature: Config PreventClassStacking #16

Open
laasker opened this issue Sep 11, 2024 · 0 comments
Open

Feature: Config PreventClassStacking #16

laasker opened this issue Sep 11, 2024 · 0 comments

Comments

@laasker
Copy link
Collaborator

laasker commented Sep 11, 2024

Describe your feature request or suggestion in detail

A config that, when is active, prevent stacking of the same class on same team.

So for example, prevents an arena composition of Warrior, Shadow Priest and Discipline Priest when active (prevent 2 priests on the same team)

Maybe having multiple option:

PreventClassStacking 0 = Off (default)
PreventClassStacking 1 = Prevent ClassStacking on all roles (melee/ranged/healer)
PreventClassStacking 2 = Prevent ClassStacking only between melees
PreventClassStacking 3 = Prevent ClassStacking only between rangeds
PreventClassStacking 4 = Prevent ClassStacking only between melee and rangeds
PreventClassStacking 5 = Prevent ClassStacking only between healer and melees
PreventClassStacking 6 = Prevent ClassStacking only between healer and rangeds

And if we can make it even harder, then add another config to only prevent ClassStacking of the configured classes, like:

PreventClassStacking.Classes =
1 - (Prevent Warrior stacking)
2 - (Prevent Paladin stacking)
4 - (Prevent Hunter stacking)
8 - (Prevent Rogue stacking)
16 - (Prevent Priest stacking)
32 - (Prevent Death Knight stacking)
64 - (Prevent Shaman stacking)
128 - (Prevent Mage stacking)
256 - (Prevent Warlock stacking)
1024 - (Prevent Druid stacking)

So if i set
PreventClassStacking = 2
PreventClassStacking.Classes = 3

Then only melee roles and paladin/warrior classes are affected, meaning its possible to get an composition of Rogue/Melee Paladin(DPS)/Healer Paladin.

Describe a possible solution to your feature or suggestion in detail

No response

Additional context

No response

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

No branches or pull requests

1 participant