Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Implement Spell damage rolls #538

Open
esandvik opened this issue Oct 17, 2023 · 3 comments
Open

Feature: Implement Spell damage rolls #538

esandvik opened this issue Oct 17, 2023 · 3 comments
Labels
enhancement New feature, request or improvement
Milestone

Comments

@esandvik
Copy link

Knight of St. Ydris uses a + to all damage
There is currently bonuses.meleeDamageBonus, bonuses.rangedDamageBonus but no bonuses.spellDamageBonus attribute
It also allows for future classes that may use a bonus to spell damage.

@esandvik esandvik changed the title Enhancement +Spell Damage Feature: Spell Damage Player Actor Attribute Oct 17, 2023
@esandvik esandvik changed the title Feature: Spell Damage Player Actor Attribute Feature: Bonus Spell Damage Player Actor Attribute Oct 17, 2023
@Muttley
Copy link
Owner

Muttley commented Oct 17, 2023

The reason for this is that currently there's no concept of spell damage at all in the system. Spells only perform success/failure rolls, damage is a manual thing.

@esandvik
Copy link
Author

Yeah I saw that today when I started digging into the system. So for now I think this can wait until that is added. But I do think that damage would be useful on the spell

@Muttley
Copy link
Owner

Muttley commented Oct 19, 2023

But I do think that damage would be useful on the spell

Yes, definitely.

@bakbakbakbakbak bakbakbakbakbak added the enhancement New feature, request or improvement label Oct 21, 2023
@bakbakbakbakbak bakbakbakbakbak added this to the to-be-done milestone Oct 21, 2023
@bakbakbakbakbak bakbakbakbakbak changed the title Feature: Bonus Spell Damage Player Actor Attribute Feature: Implement Spell damage rolls Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, request or improvement
Projects
None yet
Development

No branches or pull requests

3 participants