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
Icicle Burst can now only be played when enemy units were frozen this turn #27
Conversation
if (newState.specifics.potentialFrozenEnemiesInAColumn) { | ||
newState.specifics.potentialFrozenEnemiesInAColumn = 0 | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can skip the condition entirely here and simply reassign:
newState.specifics.potentialFrozenEnemiesInAColumn = 0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to keep it consistent with the Frozen Core test which reassigns mana only when it's not equal to 0, even though the test could be removed. It's probably better to remove both though
case 'W2': | ||
// Frosthexers | ||
newState.specifics.potentialFrozenEnemiesInAColumn += 2 | ||
break | ||
case 'W6': | ||
// Moment's Peace | ||
newState.specifics.potentialFrozenEnemiesInAColumn += 3 | ||
break | ||
case 'W11': | ||
// Midwinter Chaos | ||
newState.specifics.potentialFrozenEnemiesInAColumn += 4 | ||
break |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’m not sure how much value there is in counting enemies. Maybe we could simplify it like this:
case 'W2':
case 'W6':
case 'W11':
newState.specifics.potentiallyFrozenUnits = true
break
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was planning to implement Zhevana's ability after - that's why I'm only counting in a column
@@ -20,8 +20,12 @@ export default class DeckMechanisms extends React.Component { | |||
hand: [], | |||
RNG: 'REGULAR', | |||
hasCycledThisTurn: false, | |||
noUnitsOnFirstTurn: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good thinking to move it in specifics
. Way better. :)
Co-Authored-By: Kitty <53430374+KittySparkles@users.noreply.github.com>
No description provided.