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

added support for charge 2024 feat with additional GWF 2014/24 #1195

Merged
merged 3 commits into from
Nov 27, 2024

Conversation

dmportella
Copy link
Collaborator

updated settings
added support for 2014/24 gwf since the dmg now is addtional damage die.

Old charger feat

Screenshot 2024-11-23 154444

New Charger feat

Screenshot 2024-11-23 154521

tests

Great axe gwf 2014

should apply gwf on charger feat dice because weapon is a heavy/two-handed weapon

Screenshot 2024-11-23 155713

Great axe gwf 2014

should apply gwf on charger feat dice because weapon is a heavy/two-handed weapon

Screenshot 2024-11-23 155720

Great axe gwf 2024

should apply gwf on charger feat dice because weapon is a heavy/two-handed weapon

Screenshot 2024-11-23 155842

polearms 2014 gwf 2024

should apply gwf on charger feat dice because weapon is a heavy/two-handed weapon

Screenshot 2024-11-23 155912

polearms 2024 gwf 2024

should apply gwf on charger feat dice because weapon is a heavy/two-handed weapon

Screenshot 2024-11-23 155928

polearms 2024 gwf 2014

should apply gwf on charger feat dice because weapon is a heavy/two-handed weapon

Screenshot 2024-11-23 160613

polearms 2014 gwf 2014

should apply gwf on charger feat dice because weapon is a heavy/two-handed weapon

Screenshot 2024-11-23 160551

Non-heavy weapon

should NOT apply gwf on charger feat dice because weapon is NOT a heavy/two-handed weapon

Screenshot 2024-11-23 160756

udpated settings
added support for 2014/24 gwf since the dmg now is addtional damage die.
Copy link
Owner

@kakaroto kakaroto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small comment, looks good otherwise 👍

if(character.hasGreatWeaponFighting(2014)) {
charge_dmg += "ro<=2";
} else if(character.hasGreatWeaponFighting(2024)) {
charge_dmg = charge_dmg.replace(/([0-9]*)d([0-9]+)([^\s+-]*)(.*)/g, (match, amount, faces, roll_mods, mods) => {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need all this complex regexp, you know charge_dmg is 1d8, you can just do charge_dmg = "1d8min3" or use += safely, like for 2014 version

@dmportella
Copy link
Collaborator Author

@kakaroto change done <3

Copy link
Owner

@kakaroto kakaroto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick change!

@kakaroto kakaroto merged commit b6543a2 into kakaroto:master Nov 27, 2024
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.

2 participants