Skip to content

Remove Lich asset warning and fix JIT 2.1 Issues#1612

Merged
LocalIdentity merged 3 commits into
PathOfBuildingCommunity:devfrom
justjuangui:fix_lichdecoponse
Dec 16, 2025
Merged

Remove Lich asset warning and fix JIT 2.1 Issues#1612
LocalIdentity merged 3 commits into
PathOfBuildingCommunity:devfrom
justjuangui:fix_lichdecoponse

Conversation

@justjuangui
Copy link
Copy Markdown
Contributor

No description provided.

@justjuangui justjuangui changed the title remoive Lich decompose orbit remove Lich decompose orbit Dec 15, 2025
Missing a few cases of pairsSortByKey
Also required me to edit the dkson file to have the json object sorted by keys as it uses pairs atm resulting in the json export changing every time
Changed the export of the granted effect names to use the BaseItemType name as this is what the game uses
This fixes the new Djinn nodes having the wrong text for the command skills
@LocalIdentity LocalIdentity changed the title remove Lich decompose orbit Remove Lich asset warning and fix JIT 2.1 Issues Dec 16, 2025
@LocalIdentity LocalIdentity added the technical Hidden from release notes label Dec 16, 2025
@LocalIdentity LocalIdentity merged commit f8df1cf into PathOfBuildingCommunity:dev Dec 16, 2025
2 of 3 checks passed
@justjuangui justjuangui deleted the fix_lichdecoponse branch December 16, 2025 18:25
user-74 pushed a commit to user-74/PathOfBuilding-PoE2 that referenced this pull request Dec 16, 2025
…nity#1612)

* remoive Lich decompose orbit

* fix JIT 2.1 issue with pair

* Fix Sort order + granted effects

Missing a few cases of pairsSortByKey
Also required me to edit the dkson file to have the json object sorted by keys as it uses pairs atm resulting in the json export changing every time
Changed the export of the granted effect names to use the BaseItemType name as this is what the game uses
This fixes the new Djinn nodes having the wrong text for the command skills

---------

Co-authored-by: justjuangui <servicios@juacarvajal.com>
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

technical Hidden from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants