Mark cards you're working on in the second post of this issue.
If you can't check things off, leave a comment instead.
Once the card is committed, check off the card here to show that it's finished.
Don't worry about adding new spoilers to this list, it's generated by a script.
Click to expand
All cards have been implemented.
I'd like to help with this if you don't mind (or does your second post mean you're already working on all these cards?) I don't think I have permissions to check stuff off on the list but I'm about to send a pull request for Absorb Identity.
Totally, feel free to contribute. I鈥檝e been busy so I haven鈥檛 really gotten into these yet.
I'm taking Fire Giant's Fury next (saw the list updating so wanted to make sure we're not taking the same cards.)
Yeah, no problem. The help is much appreciated.
I'll take Thornmantle Striker unless you want it.
It's all yours.
Lots of new cards spoiled today. Is there a script we can run to fetch the card data so Utils/gen-card.pl will work for them?
I also just sent a PR implementing the Boast mechanic as a common ability class so that we can use that for the new cards.
I have a personal script I use but I'm waiting on scryfall to update with all the new stuff. Usually I don't implement a new mechanic until I'm implementing a card that uses it.
I'm planning on implementing Varragoth, Bloodsky Sire which uses Boast. I just don't know the formatting for Utils/mtg-cards-data.txt which is why I was asking about a script. I guess it's probably best to wait on scryfall?
If you want to implement it now you can, but scryfall should be caught up by the end of the day. The formatting is like this:
Card Name|Set Name (Full)|Collector Number|Rarity (C/U/R/M)|Mana Cost|Type Line|Power|Toughness|Rules text with $ denoting line breaks|
If it's a planeswalker then replace |Power|Toughness| with |Loyalty|. Also don't worry about having multiple versions, those will get added later.
Thanks. I've got it implemented now. I'll do the snow duals too.
Just implemented Sigrid. I'm no expert on git so before I merge the PR I was wondering which procedure you guys prefer when merging them (squash & merge, rebase, merge commit).
I'm going to take Calamity Bearer.
@ciaccona007 depends on PR's commits:
P.S. If you want PR's review then don't merge it -- wait a new review from another dev. If you ensure about fix or card then you can add it to master without PR.
Taking Esika, God of the Tree next.
Taking Koll, the Forgemaster.
I've got Tyvar Kell mostly done. I'm just missing the haste ability on the emblem which is worded strangely. Just from a Magic rules perspective, I'm not sure how giving an Elf on the stack haste makes sense.
I'm thinking I may have to give the emblem a static ability so it can give haste once the Elf has actually entered the battlefield (after checking that it was cast) but if anyone has a better idea, let me know.
I'll take another look at it tomorrow and try to get it working.
You can probably generate a continuous effect that applies to it on the stack and on the battlefield, you just need to check two possible zone change counter values.
I'll take Kolvori, God of Kinship.
CosimaGodOfTheVoyage.zip
Withercrown
https://github.com/magefree/mage/commit/708a486eb65213d5f851bf3aa49222edd19a86e4
Wings of the Cosmos
https://github.com/magefree/mage/commit/72c4369899d8ad3fbcf2c5c75649f7087259c143
Valki, God of Lies
Done
Tergrid, God of Fright
Done
Alrund, God of the Cosmos
Done
Valor of the Worthy
Done
Foretell keyword ability.
Implemented. Please check for any bugs yet again in the new update.
Cosmos Charger -> The cost reduction is easy, but perhaps someone knows of a way to set the ActivationStatus to True on the Special Action of Foretelling.
Cosima, God of the Voyage : I put the current code for the card. I have moved on. If anyone can fix it, please have at it.
Dream Devourer: Working on this now.
I'll take Battle of Frost and Fire.
Taking Draugr's Helm next.
I'm taking Firja's Retribution.
@jeffwadsworth You probably need to clear the database if cards don't show up. I have a script that deletes the Mage.Client/db and Mage.Server/db folders that I run before I test a new card.
@weirddan455 server/client/tests runs a new cards checking on startup, so it will auto-add new card to database. DB cleaning can be useful only for manual checking after a text changing in existing card.
More tips:
VerifyCardDataTest->test_showCardInfo -- it's very fast and don't use db, so you can see actual card's info all the time;FAST_SCAN_WITHOUT_DATABASE_CREATE param (search it in code);Yeah it's good to write unit tests whenever you can, even though I don't do it nearly enough.
@jeffwadsworth you should be able to use the same code as Leonin Shikari for Cosmos Charger.
@jeffwadsworth you should be able to use the same code as Leonin Shikari for Cosmos Charger.
Sadly, that doesn't work either. It doesn't affect the ActivationStatus.
Just saw you pushed the snow land collation. Is there something we need to do to enable booster draft for the set seeing as it's mostly complete now?
I don't think so but I don't actually know for sure
@theelk801 OK, it seems to be working now. I thought I had tried it a week or 2 ago and KHM packs weren't an option but maybe I just missed it.
I'll take Cosima
Thanks to everyone who helped get this done.
Visibility of MDF card's counters was fixed in c381ea77168c382c365de197d670aae7051b9ba4 (Cosima, God of the Voyage), so it can be closed.
Most helpful comment
Thanks to everyone who helped get this done.