Add support for permanent mercenaries - #10320
Draft
ZenJediMind wants to merge 1 commit into
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #9984
Description of the problem being solved:
Permanent mercenaries were not hireable in PoB.
Steps taken to verify a working solution:
Link to a build that showcases this PR:
https://pobb.in/OMeTtl_4nU_A (https://www.reddit.com/r/pathofexile/comments/1w4qcl0/int_stack_crit_doryani_storm_call_luminary/)
https://pobb.in/N0AmQJyjlVv7 (Aura Bot Destructive Link)
https://pobb.in/caygE1zcthMx (Rathpith KBoC Flame Link)
https://pobb.in/Ia5XhXfLtq7x (Manyshot Flame Link)
Before screenshot:
After screenshot:
Controversial changes
As this PR is quite broad and touches a lot of PoB, it has a large number of changes across a lot of systems. These changes are what I view as the most controversial.
What: Config tab now has an actor selector. An actor is either the player or the mercenary.
Why: Configuration of Merc vs Player can be different (Though they do share the same enemy).
What: Items tab, item set selection, is now just changing the view, not which set is actually equipped.
Why: To support trade for these items and the trade query, we need to be able to select which item set to search for, and know which sets are currently equipped. The config tab now holds which item set each actor is wearing.
AI Usage Disclosure: Generative AI was used in the development of this.