Spells are cheap, powerful, and ruin bosses. Players are also not told when they unlock a spell.
What is the new behavior?
Spells are less cheap, less powerful, and don't always effect bosses (unless it's a critical hit)
The player, on levelling up to the level required to unlock a spell, will see that information in the level up dialog, as well as an entry in their journal.
PR Checklist
Please check if your PR fulfills the following requirements:
[x] Tested code with current upstream repo
[ ] Added new tests to the test suite for the feature
[ ] Tested the full test suite, and ensured that the feature does not break current build.
[ ] Docs have been added/updated which fit (for bug fixes / features)
[x] Contains NO breaking changes
[x] Associated with an issue (GitHub or internal)
[ ] Checked licensing of Frameworks/Libraries (if applicable)
Other information
This is based off of the branch for #273, so that should probably be merged first.
GitHub Issue (If applicable): #288, #286, #279
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Spells are cheap, powerful, and ruin bosses. Players are also not told when they unlock a spell.
What is the new behavior?
Spells are less cheap, less powerful, and don't always effect bosses (unless it's a critical hit) The player, on levelling up to the level required to unlock a spell, will see that information in the level up dialog, as well as an entry in their journal.
PR Checklist
Please check if your PR fulfills the following requirements:
Other information
This is based off of the branch for #273, so that should probably be merged first.
Internal Issue (If applicable): closes #288, closes #286, closes #279