sirrus233 / twilight-imperium-helper

Game assistant for Twilight Imperium
1 stars 0 forks source link

Fighter II checkbox #3

Closed chelseaSchmidt closed 1 month ago

chelseaSchmidt commented 1 month ago

Took a stab at this, check my logic though!

New budget inputs:

New budget display field:

I realized there should also be a checkbox for "I'm building in a space-only system" for Saar (maybe others?), which would trigger ground forces to be subtracted from shipCapacityRemaining, but leaving that for another PR!

chelseaSchmidt commented 1 month ago

@sirrus233 Aaah been out all day and just had time to check this out, these review comments are so on point, dude thank you!! Gonna see if I can get a jump on these before we call tomorrow!

chelseaSchmidt commented 1 month ago

@sirrus233 I think I got everything, you want to take one more look?

Each time I change any of the math I wish for unit tests, and yet am too lazy to set up Jest. I will simply complain about it endlessly

sirrus233 commented 1 month ago

6