Bombing Bonus Not Working
-
I have a Bomber that gets 2 Die rolls on Attack and Defense. It also has a "Bombing Bonus" of +2. The Bonus is not working.
Shouldn't be able to roll a 1. The Bombers that only have 1 Die roll work correctly.
@wc_sumpton @victoryfirst any ideas on this ? My only guess is that somehow the
<option name="bombingBonus" value="2"/>
isn't working with the
<option name="attackRolls" value="2"/>
<option name="defenseRolls" value="2"/> -
Did this used to work before? Or did you try something and that ended up not working?
-
Hi Victory
Idk. I never noticed it until a couple weeks ago. We've had the Hvy Bmbrs for EXP probably 4-5 years now. I think I'd have noticed it but ...
I'm still on the first update 15640
-
Lol this looks cool xD
-
heh heh Joe Pants image
I got it automated for the big targets. Up to 3 different bombs so after you fly the bomb in it'll auto change to the cloud in NCM lol
-
I just tried it myself and I got the bonus:
-
It looks like the bonus is only applied to the first roll.
-
Can you create a GitHub issue? It's an engine thingy that is not too hard to fix.
-
-
Lol it looks like the first roll gets the bonus added up but the second roll gets it subtracted
dice rolls: [2, 2] die 2 dice + bonus: [4, 0] die 2 dice + bonus: [4, 0] nextDieIndex: 1 -
Nope, it is that the die index does not update correctly so the first roll gets replaced with the second roll, and the second number always stays 0.
dice rolls: [3, 0] dice + bonus: 5 dice array [0, 0] die index 0 dice + bonus: 2 dice array [5, 0] die index 0Notice how die index stays 0. I didn't print this, but dice array would probably look like [2,0] which corresponds to a roll of 3 and 1 in-game. That corresponds with what the game shows.
-
-
Nah I already fixed it. Will create a PR soon.
-
-
So it's OK to do the update after it merges ? I think WC the only one with the button issue. Don't want any of the guys to get that
-
@beelee I thought you did not get the button issue, You aren't running on the latest version? I don't think you will get the button issue if you update, I am not getting it either on the latest.
-
-
@beelee probably failed spotlessCheck lol
-
heh heh exit code 1
Hello! It looks like you're interested in this conversation, but you don't have an account yet.
Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.
With your input, this post could be even better 💗
Register Login