feat: Event triggers #1166
Open
feat: Event triggers #1166
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Great idea! I'll have a look when I have the time. |
|
Also, if you don't mind I'll just look into this after #1169 is merged. |
|
No problem, I'll rebase once #1169 is merged. |
UberWaffe
force-pushed
the
feature/condition_fire_ruins
branch
4 times, most recently
from
9be117d to
f71a717
Compare
|
@crudelios Rebased, incorporated the condition group and other changes, and tested. |
UberWaffe
force-pushed
the
feature/condition_fire_ruins
branch
from
f71a717 to
302c1d0
Compare
|
Rebased after c2c958b (Fix importing of malformed event xml files). |
|
Oops, new recent conflict in translation file! Can you fix so I check this please? |
Added event trigger for building caught fire Added event trigger for building collapse poor maintenance
UberWaffe
force-pushed
the
feature/condition_fire_ruins
branch
from
302c1d0 to
673162c
Compare
|
@crudelios Rebased on latest |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added event trigger for building caught fire
Added event trigger for building collapse poor maintenance
This allows for events to trigger their checks on events other than the start of a new game month.
Currently added trigger types include: Building catching fire, building being constructed, building deleted by player, etc.
Also adds the concept of 'context'.
For these new types of triggers, this is the type of building that is involved.
I.e. You can do things like fire off an event when the player places a specific type of building.
Or when a specific type of building (or even any type) catches fire.
Should support redoing the tutorials completely with events only.