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

[Magiclysm] Fix animist convert spell missing damage type #78621

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

ave-
Copy link
Contributor

@ave- ave- commented Dec 17, 2024

Summary

Bugfixes "Fix animist Convert spell missing damage type"

Purpose of change

Convert spell causes error because it misses damage type.

Describe the solution

Setting damage type to "pure" prevents any resistances decreasing damage.

Describe alternatives you've considered

Setting damage to any other type if PERCENTAGE_DAMAGE ignores resistances.

Testing

Casting Convert does not cause error anymore.

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mods Issues related to mods or modding Mods: Magiclysm Anything to do with the Magiclysm mod Mechanics: Enchantments / Spells Enchantments and spells labels Dec 17, 2024
@github-actions github-actions bot requested a review from KorGgenT December 17, 2024 09:34
@ave- ave- changed the title Fix animist convert spell missing damage type [Magiclysm] Fix animist convert spell missing damage type Dec 17, 2024
@github-actions github-actions bot added <Bugfix> This is a fix for a bug (or closes open issue) 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 Dec 17, 2024
@Maleclypse Maleclypse merged commit 109a8a0 into CleverRaven:master Dec 18, 2024
27 checks passed
@ave- ave- deleted the magiclysm_convert_fix branch December 18, 2024 21:53
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 <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mechanics: Enchantments / Spells Enchantments and spells Mods: Magiclysm Anything to do with the Magiclysm mod Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants