Skip to content

Trader tool PoB1 improvement ports and PoE2 specific fixes#1801

Draft
vaisest wants to merge 18 commits intoPathOfBuildingCommunity:devfrom
vaisest:trader-tool-port
Draft

Trader tool PoB1 improvement ports and PoE2 specific fixes#1801
vaisest wants to merge 18 commits intoPathOfBuildingCommunity:devfrom
vaisest:trader-tool-port

Conversation

@vaisest
Copy link
Copy Markdown

@vaisest vaisest commented Apr 18, 2026

Fixes:

#469 might be related

Description of the problem being solved:

  • Ports PoB1 trader improvements
  • Fixes mod weight generation, where for example some local modifiers were not correctly generated as PoB does not internally use the (local)
  • Fixes radius jewels and improves the jewel search in general by removing the any option, which would have caused poor results due to the lack of available filters
  • Uses OAuth header instead of poesessid. Handling logins in different view is solved by putting the PoEAPI instance in main. Hopefully this is fine?
  • Uses poe.ninja currency page endpoint for currency data. (This could be ported to pob1 for consistency too)

Steps taken to verify a working solution:

Possible issues

I had problems with 429 rate limiting. I'm not sure if that's because of me mashing f5 to test changes, or a real issue.

Link to a build that showcases this PR:

Nothing specific, but:

Before screenshot:

After screenshot:

Depends on #1798.

@vaisest vaisest marked this pull request as ready for review April 20, 2026 10:53
@vaisest vaisest marked this pull request as draft April 20, 2026 13:42
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