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

Applying scrolls to role groups #7

Open
sudojunior opened this issue Nov 7, 2022 · 1 comment
Open

Applying scrolls to role groups #7

sudojunior opened this issue Nov 7, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@sudojunior
Copy link
Member

Given the logic needed to compute what slots should be filled, and how they should behave. There is... logical reasoning to determine that the individual selection of role groups can be done after players have been handed their roles... then again, there's a possibility that it's an all Any game, and the chances won't matter (more so that it would be critical to get unique roles correct before the game sequence is initiated... and to store the initial result in a database, then retrieve it and rebuild the resulting role data... just to resolve the groups).

While it does sound reasonable for a method to hide the chosen roles, it is more likely that an attribute can be added stating that it was an all any role, and what it originated from - therefore maintaining the opportunity to hide a chosen role if required.

@sudojunior sudojunior added the enhancement New feature or request label Nov 7, 2022
@sudojunior
Copy link
Member Author

This also introduces a new opportunity to decide if the randomly selected scrolls should target a role or a group, and another opportunity to compound the effect - which it can already do, it would just need to accept the new parameters for groups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant