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

Fix damage rolls not dealing atleast 1 damage #17414

Closed
wants to merge 5 commits into from

Conversation

MiaOjala
Copy link

closes #16484

@stwlam
Copy link
Collaborator

stwlam commented Nov 21, 2024

I'm not sure we want to set it to be at least 1 per instance.

@MiaOjala
Copy link
Author

Any idea which direction this should be approached from then? check for it later or make it an option perhaps?

@stwlam
Copy link
Collaborator

stwlam commented Nov 22, 2024

It should probably just ensure that it's at least zero for each instance. Then the IWR application helper can pre-bump the damage to 1 if the summed non-persistent-damage instances start at zero.

@MiaOjala
Copy link
Author

It seems that if IWR pre-bumps the damage to 1 it is only used when the actor has immunities/resistances to it. in the case of nothing and weaknesses the increase is ignored.

@MiaOjala MiaOjala reopened this Nov 23, 2024
@MiaOjala
Copy link
Author

It seems someone has been making something larger that could also fix this problem, but the code seems unfinished so this would be a smaller fix

@MiaOjala
Copy link
Author

Hello! Could it be possible for someone to check this out?

@MiaOjala MiaOjala closed this Nov 28, 2024
@CarlosFdez
Copy link
Collaborator

Sorry, the only person who can reasonably give you a good answer regarding IWR is stwlam, and I imagine he's very busy with V13 foundry dev at the current moment.

@MiaOjala
Copy link
Author

Alright thanks for the answer. I'll leave this here and make another fork for the more complicated way of approaching this.

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.

Damage Rolls are not dealing a minimum of 1 damage
3 participants