Skip to content

Fix trader parsing everything as explicits - #10073

Merged
Wires77 merged 5 commits into
PathOfBuildingCommunity:devfrom
vaisest:fix-trader-item-parsing
Aug 1, 2026
Merged

Fix trader parsing everything as explicits#10073
Wires77 merged 5 commits into
PathOfBuildingCommunity:devfrom
vaisest:fix-trader-item-parsing

Conversation

@vaisest

@vaisest vaisest commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description of the problem being solved:

This fixes item parsing for the trader. The problem is caused by GGG's provided item string having a weird format, and this instead builds a new string similarly to what PoB2 does.

Scourge mods not added here because I don't know how they work, they have a weird format, and they don't seem to be supported properly until #10039 is merged.

Depends on #10026

Steps taken to verify a working solution:

  • Items seem to parse correctly

Link to a build that showcases this PR:

N/A

Before screenshot:

(everything parsed as explicits)

image

After screenshot:

image

@vaisest
vaisest force-pushed the fix-trader-item-parsing branch from 9f0861e to fbc16b0 Compare August 1, 2026 09:19
@Wires77 Wires77 changed the title Fix trader item parsing Fix trader item parsing everything as explicits Aug 1, 2026
@Wires77 Wires77 changed the title Fix trader item parsing everything as explicits Fix trader parsing everything as explicits Aug 1, 2026
@Wires77 Wires77 added the bug: accuracy Wording differences label Aug 1, 2026
@Wires77
Wires77 merged commit e72f082 into PathOfBuildingCommunity:dev Aug 1, 2026
3 checks passed
@vaisest
vaisest deleted the fix-trader-item-parsing branch August 23, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug: accuracy Wording differences

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants