Skip to content

Add talisman enchants to trader - #10078

Open
vaisest wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
vaisest:talisman-trader
Open

Add talisman enchants to trader#10078
vaisest wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
vaisest:talisman-trader

Conversation

@vaisest

@vaisest vaisest commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description of the problem being solved:

Talisman implicits became enchants and thus weren't included in QueryMods.lua. This exports the enchant mod ids and adds them to QueryMods.lua. I also changed the talisman checkbox to default to off, because otherwise the search is completely useless. It could also be a good idea to separate amulet and talisman completely so that the amulet implicits and talisman enchants never appear in the same search, but the explicit mods seem to be the same so I'm not sure that makes a lot of sense.

Steps taken to verify a working solution:

  • Enchants appear in searches
  • No real runtime logic changes so there shouldn't be any problems.

Link to a build that showcases this PR:

https://poe.ninja/poe1/pob/94aa6

Before screenshot:

After screenshot:

image image

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