Skip to content

Fix Holy Strike Minion missing Lightning damage conversion from skill gem#9578

Merged
LocalIdentity merged 2 commits intoPathOfBuildingCommunity:devfrom
zekyriah:dev
Mar 11, 2026
Merged

Fix Holy Strike Minion missing Lightning damage conversion from skill gem#9578
LocalIdentity merged 2 commits intoPathOfBuildingCommunity:devfrom
zekyriah:dev

Conversation

@zekyriah
Copy link
Copy Markdown
Contributor

Fixes # .

Description of the problem being solved:

Holy strike on the current release build does not convert 50% of the minion damage to lightning damage (as shown in the calcs tab when showing minion stats).

Steps taken to verify a working solution:

  • Add statMap to ensure the modifier is respected
  • Use build exhibiting the issue on release version to confirm it is fixed on branch version

Link to a build that showcases this PR:

https://pobb.in/iFpLVAuWKm87

Before screenshot:

image

After screenshot:

image

@LocalIdentity LocalIdentity changed the title Fix Holy Strike minion damage conversion calculation Fix Holy Strike Minion missing Lightning damage conversion from skill gem Mar 11, 2026
@LocalIdentity LocalIdentity added the bug: calculation Numerical differences label Mar 11, 2026
Copy link
Copy Markdown
Contributor

@LocalIdentity LocalIdentity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed the PR by adding the required lines to the act_str.txt file.
The lua files are generated from the txt files using the export tool

@LocalIdentity LocalIdentity merged commit d01b88a into PathOfBuildingCommunity:dev Mar 11, 2026
3 checks passed
NDari pushed a commit to NDari/PathOfBuilding that referenced this pull request Mar 31, 2026
… gem (PathOfBuildingCommunity#9578)

* Fix Holy Strike minion damage conversion calculation

* Add export lines

---------

Co-authored-by: LocalIdentity <localidentity2@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug: calculation Numerical differences

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants