Skip to content

Add support for Sinister Jewel sockets#2277

Open
followingthefasciaplane wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
followingthefasciaplane:dev
Open

Add support for Sinister Jewel sockets#2277
followingthefasciaplane wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
followingthefasciaplane:dev

Conversation

@followingthefasciaplane

Copy link
Copy Markdown

Description:

intended to add support for sinister jewel sockets granted by Voices and Zarokh's Gift. granted sockets now apply in calculator environments, update main build socket state only during the main calc pass, and are removed cleanly when the granting anoint/item goes away.

also wanted to fix the anoint picker so Zarokh's Gift appears by adding the new 0.5 emotions to the filter list, and verify tooltip comparisons include jewels socketed.

dynamically adds or removes socketables from the items UI with persistent socket allocation

Steps taken to verify a working solution:

  • test spec can be found in commit
  • preliminary testing allocating and unallocating sinister sockets
  • needs a more experienced contributor to review, im not entirely confident

Screenshots

z6 z5 z4 z3 z2 z1

adds support for sinister jewel sockets granted by amulet anoints and Voices, including Zarokh's Gift. granted sockets now apply in calculator environments, update main build socket state only during the main calc pass, and are removed cleanly when the granting anoint/item goes away.

also fixes the anoint picker so Zarokh's Gift appears by adding the new 0.5 emotions to the filter list, and verifies tooltip comparisons include jewels socketed into prospective Zarokh sockets.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant