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

[MoM/XE] Add deactivate_conditions to recurring EoCs that don't need to recur for everyone #78161

Merged
merged 2 commits into from
Nov 28, 2024

Conversation

Standing-Storm
Copy link
Contributor

@Standing-Storm Standing-Storm commented Nov 26, 2024

Summary

Mods "[MoM/XE] Add deactivate_conditions to recurring EoCs that don't need to recur for everyone"

Purpose of change

My save crashed and while seeing if EoCs were causing it I saw a few recurring EoCs in the save that don't need to be there and could use some deactivate_conditions

Describe the solution

Add deactivate_conditions to various mod EoCs. These are things like EOC_INVENTOR_RECIPE_GAIN (gives you your first Inventor recipe in XE, no one but an Inventor will ever need it) or EOC_CLAIR_RAD_SENSE_OUTSIDE_RECURRING (part of a Clairsentient power in MoM, not necessary for anyone who isn't a Clairsentient).

All recurring EoCs are turned back on if you gain a new trait, so I made sure the deactivate conditions were all trait-based.

Describe alternatives you've considered

Testing

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding EOC: Effects On Condition Anything concerning Effects On Condition Mods: Xedra Evolved Anything to do with Xedra Evolved Mods: Mind Over Matter labels Nov 26, 2024
@github-actions github-actions bot requested a review from Maleclypse November 26, 2024 03:54
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Nov 26, 2024
@Night-Pryanik
Copy link

@Standing-Storm could you please resolve conflicts?

@Night-Pryanik Night-Pryanik merged commit 75431d4 into CleverRaven:master Nov 28, 2024
17 of 23 checks passed
@Standing-Storm Standing-Storm deleted the homullus-deactivate branch November 28, 2024 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions EOC: Effects On Condition Anything concerning Effects On Condition [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mods: Mind Over Matter Mods: Xedra Evolved Anything to do with Xedra Evolved Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants