-
Notifications
You must be signed in to change notification settings - Fork 105
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
NEW: EndInvestigatorTurn Trigger #1110
Comments
While I agree with your point, there is a subtle use to these. If more than one event triggers from EndRound it will randomly pick one to run. Using EndRound then StartRound guarantees an order of those two events, though you could argue that using one EndRound triggered event with multiple events chaining from it is a more elegant way to guarantee event orders (and make them scalable past two events). Also if EndRound triggers on #round ==2 then technically StartRound will trigger in #round == 3. So it would make it easier to keep track of what rounds you expect things to happen in. I would remove it (or combine it) as it will break legacy scenarios. Rather adding additional triggers may make more sense. I think the following triggers would be helpful -
|
@NPBruce brought up an interesting point on Discord and that EndRound is used for Descent. |
I know this may not be the most urgent ticket, but if it is not too much trouble, I really would LOVE to get a trigger on EndInvestigatorTurn. |
Description of Problem or Question
The trigger StartRound gets activated at the start of the round. Investigators do their actions. When pressing End-turn Mythos happens, Monsters happen and then the EndRound trigger becomes active, and the the new round starts and StartRound is activated.
Sind EndRound (1) is immediately followed by StartRound(2) the triggers are actually the same from the view of the scenario writer.
What I miss most is a trigger that marks: EndInvestigatorTurn.
I can also imagine an EndMythos but I will create a seperate ticket for that since I miss that one less but can imagine some nice uses for it :). (edit: see #1111)
Valkyrie Version
2.2.04
The text was updated successfully, but these errors were encountered: