Skip to content

Conversation

@NaimSantos
Copy link
Contributor

No description provided.

@NaimSantos NaimSantos marked this pull request as ready for review March 27, 2025 19:37
@NaimSantos NaimSantos requested a review from pyrQ as a code owner March 27, 2025 19:37
pyrQ and others added 28 commits April 10, 2025 10:53
added hint on the protection effect
-  it was missing a hint message for the target part
- it was making the player select the effect before targetting
- use only one function for the resolution
originally those resets didn't include RESET_LEAVE and RESET_TOFIELD (the monster is still in the ED)
- use AddMustFirstBeFusionSummoned
- use GetBinClassCount
it's "Must be", not "Must first be"
- Use Duel.IsExistingTarget instead of Duel.IsExistingMatchingCard.
- Missing listed_names.
- Save the monster's name on the field before Tributing it for the cost.
- Added chkc handling.
- Added handling to prevent applying just the effect without paying the cost.
- Use RESETS_STANDARD_DISABLE_PHASE_END for the ATK boost.
- Check that there are enough cards on the field to destroy to not allow Tributing more causing the number of cards on the field to be fewer than the number Tributed.
- Don't destroy anything if there are fewer cards on the field than the number Tributed.
- Cannot apply the effect without paying the cost.
Declare the card name before targeting.
- Can target a monster on either field.
- Fixed hint message.
- Added cannot disable flags to the ATK changing effects.
Fixed an issue where the only effect you could activate was the Special Summoning one but you were allowed to target a "Utopia" Xyz that didn't have a corresponding one in the Extra Deck that could be summoned.
missing parameters
@NaimSantos NaimSantos merged commit fff476a into master Apr 18, 2025
1 check passed
@edo9300 edo9300 deleted the updates branch April 19, 2025 08:30
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.

3 participants