PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
CWDT supports skills with higher levels
Open
Nobody has claimed this yet.
enhancement
wontfix
- Dominant language
- Lua
- Stars
- 5.4k
- Forks
- 2.4k
- Avg merge
- 1d 12h
- Merged PRs (30d)
- 26
Description
Check version
- I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
- I've checked for duplicate open and closed issues by using the search function of the issue tracker
What platform are you running Path of Building on?
Windows
How is Path of Building expected to behave?
When a spell is higher level than CWDT threshold, it should not be supported by trigger and its mana cost modified
How does Path of Building behave?
CWDT supports everything in the same skill group
How to reproduce the issue
put lvl1 CWDT and lvl20 spell gem into a same skill group
Character build code
https://pobb.in/wq9rJr8Lx_w6
Screenshots
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce the issue with a level 1 CWDT and level 20 spell gem in the same skill group, using the provided build code as a reference. Trace the CWDT support calculation and verify that spells above its level threshold are not supported and do not have their mana cost modified.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- lua
- Domain
- game-dev
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 35/100