Skip to content

Fix Warden import crash - #10002

Merged
LocalIdentity merged 1 commit into
PathOfBuildingCommunity:devfrom
vaisest:fix-warden-import
Jul 25, 2026
Merged

Fix Warden import crash#10002
LocalIdentity merged 1 commit into
PathOfBuildingCommunity:devfrom
vaisest:fix-warden-import

Conversation

@vaisest

Copy link
Copy Markdown
Contributor

Fixes importing wardens partly caused by my Oauth PR. There were two problems:

  • import uses name, ascend name map uses id (despite the variable name)
  • legacy alt ascendancy would remove "warden" anyway

Description of the problem being solved:

Steps taken to verify a working solution:

Link to a build that showcases this PR:

https://poe.ninja/poe1/builds/standard/character/joseph830411-4621/AirWaveFact?i=0&search=class%3DWarden

Before screenshot:

After screenshot:

@LocalIdentityLocalIdentity changed the title Fix warden import crashFix Warden import crashJul 25, 2026
@LocalIdentity
LocalIdentity merged commit 9613635 into PathOfBuildingCommunity:devJul 25, 2026
3 checks passed
@LocalIdentityLocalIdentity added the crash Causes PoB to crash and is High Priority label Jul 25, 2026
vaisest added a commit to vaisest/PathOfBuilding that referenced this pull request Jul 26, 2026
DrakenKor pushed a commit to DrakenKor/PathOfBuilding that referenced this pull request Jul 27, 2026
vaisest added a commit to vaisest/PathOfBuilding that referenced this pull request Jul 28, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crashCauses PoB to crash and is High Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@vaisest@LocalIdentity