mage icon indicating copy to clipboard operation
mage copied to clipboard

[NCC] [WIP] Implement Henzie "Toolbox" Torre

Open Alex-Vasile opened this issue 2 years ago • 5 comments

Closes #8823.

Alex-Vasile avatar Sep 04 '22 18:09 Alex-Vasile

~~I think I will have to solve #9491 as part of this. Using that common effect seems like the way to go. Though I don't yet have an elegant way to get around the fact that the BlitzAbility constructor requires the Card (or at least the Card's name).~~

Alex-Vasile avatar Sep 09 '22 16:09 Alex-Vasile

given that this is the last card in the set, do you mind if I add on to this PR with all the art variants? I just wanna have them added in

theelk801 avatar Sep 23 '22 01:09 theelk801

given that this is the last card in the set, do you mind if I add on to this PR with all the art variants? I just wanna have them added in

Not at all. Though I would really appreciate some help on implementing it 😅

Alex-Vasile avatar Sep 23 '22 01:09 Alex-Vasile

yeah I think I owe you that much given that you did the rest of the set

theelk801 avatar Sep 23 '22 01:09 theelk801

yeah I think I owe you that much given that you did the rest of the set

😂 I'm not keeping track. You do plenty of other cards. I just don't know what to do next and want to wrap it up so I can finish off the rest of my leftover issues and PRs

Alex-Vasile avatar Sep 23 '22 01:09 Alex-Vasile

@Alex-Vasile I'm going to try to finish this and also fix #8885

awjackson avatar Oct 23 '22 20:10 awjackson

@Alex-Vasile I'm going to try to finish this and also fix #8885

Please!

Alex-Vasile avatar Oct 23 '22 20:10 Alex-Vasile

This may benefit from a fresh look with #8885 merged.

Notshauna avatar Jun 23 '23 07:06 Notshauna

Still test failures:

[ERROR]   HenzieToolboxTorreTest.commanderCastDiscount:79->CardTestPlayerAPIImpl.execute:283 Can't find ability to activate command: Cast Aerathi Berserker with Blitz

[ERROR]   HenzieToolboxTorreTest.gainsBlitz:63->assertBlitzed:36 Permanent should have haste expected:<true> but was:<false>

xenohedron avatar Jun 24 '23 03:06 xenohedron

@Grath do you think #9835 could help with this? Not sure the status of it

xenohedron avatar Jun 24 '23 04:06 xenohedron

I think that solving this might also provide a solution for 9835.

Grath avatar Jun 24 '23 04:06 Grath