Skip to content

Add Wickerman item data and character import support - #10455

Open
AdamZ-8113 wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
AdamZ-8113:submit/wickerman-character-import
Open

AdamZ-8113 wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
AdamZ-8113:submit/wickerman-character-import

Conversation

@AdamZ-8113

Copy link
Copy Markdown
Contributor

Description of the problem being solved:

This adds support for importing Viscera from a character into PoB. This adds their item definitions and imports them into the four Wickerman equipment slots, keeping their names and modifiers when saving, reopening or reimporting a character.

** Dependencies**

This only works with the OAuth import route which uses the Character API. Import by Account name doesn't work because the website character viewer doesn't currently expose this data.

The four captured viscera have been checked against live data. Other definitions come from the preview spreadsheet and remain provisional. Greater variants and the three Artificial Appendages have synthetic test coverage but still need live examples. Ravenous Skull's corpse-consumption recovery text is imported, but its effect isn't calculated yet.

Steps taken to verify a working solution:

  • Imported all four attached viscera from a live character response and confirmed their slots, names and modifiers survived reimport and save/load.
  • Used three synthetic Artificial Appendages with different IDs and modifiers to check that reimport and save/load preserve them as separate items.
  • All 9 import tests passed.
  • Tested the combined setup with the PRs listed above: 694 tests passed
  • Compared in game character sheet against PoB and confirmed life and armour bonuses from my Viscera are working

Link to a build that showcases this PR:

Before screenshot:

image

After screenshot:

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