sumpfork / dominiontabs

Divider generator for the card game Dominion and its expansions.
http://domdiv.sandflea.org
Other
123 stars 126 forks source link

Rising Sun, Marchland, 2nd Edition Cornucopia & Guilds #498

Closed nickv2002 closed 1 month ago

nickv2002 commented 9 months ago

FYI and future TODOs that new cards are incoming in the next few months.

nickv2002 commented 8 months ago

Looks like we're getting a 2nd edition of Cornucopia & Guilds in march and the cards have already been revealed: https://wiki.dominionstrategy.com/index.php/Cornucopia_%26_Guilds

I'll update the title of this issue to reflect that.

thale13 commented 8 months ago

Any updates on this? Just wondering. Willing to help if you can point me in the right direction.

nickv2002 commented 8 months ago

@thale13 I imagine we'll fold the promo and Corn/Guild 2nd Ed. updates into a Rising Sun update once those cards are known (that's how it's worked in the past). Not sure when that will be but probably soon. Watch the Dominion Wiki for more info on that.

If you do want to get started you can add cards to the JSON card DB, eg Marshland would get a new entry somewhere around here.

dbgalur commented 6 months ago

This second edition / upgrade combines the Guilds and Cornucopia sets into a single set. So we can talk about cards that were removed from the 1st edition of each, but the new cards are added to a combined set, rather than to a second edition of each individual set. How would you represent the upgrade set in sets_db.json? Specifically, "upgrades" expects a string rather than an array, no?

dbgalur commented 6 months ago

Actually, that may not be a relevant question. 5 of the new cards are clearly part of Cornucopia, while the other 3 new cards are part of Guilds, even if they are not explicitly marked with different icons.

nickv2002 commented 6 months ago

We obviously try to match what has been physically printed because they're dividers for physical cards. So we'll have an upgrade set and a second edition set and a first edition set (see previous upgrade packs here). The upgrades and second edition will be for the dual pack, but the first edition remains separate.

dbgalur commented 6 months ago

Okay, so I've made what I think are the necessary changes to card_db_src/cards_db.json, card_db_src/sets_db.json, card_db_src/types_db.json, card_db_src/xx/cards_xx.json, and card_db_src/xx/sets_xx.json.

But I have close to 0 experience doing stuff with git / github, and have not succeeded in running the tests on Windows, so would love any assistance someone feels like they can offer. (I suppose in the worst case I can just share the changes with someone and let them do the merge/commit/pull whatever the terminology is).

dbgalur commented 6 months ago

I've successfully managed to build/test the additions. Now all that remains is to figure out how to commit the changes and propose a change...tomorrow.

nickv2002 commented 6 months ago

Thanks for your work! If you want to learn Git and GitHub there are lots of tutorial videos on YouTube (I don't have any specific recommendations). I also think the GitHub Desktop app is a good tool for both beginners and pros so I'd recommend starting with that.

You're also welcome to zip up the json files and upload them to this thread if learning Git wasn't on your weekend list. 😆

dbgalur commented 6 months ago

I think I've created a pull request?

I worked with git maybe about a decade ago, but never with GitHub..

nickv2002 commented 6 months ago

Thanks @dbgalur

Now I just hope that Rising Sun is revealed soon!

sumpfork commented 6 months ago

Cornucopia and Guilds 2nd edition is packaged and deployed, thanks!

md5i commented 5 months ago

My attempt at adding Marchland is here: https://github.com/sumpfork/dominiontabs/pull/524

nickv2002 commented 3 months ago

All Rising Sun cards have now been revealed. I'm traveling at the moment so no help for adding them but figured I'd note it as I saw the news!

tphinney commented 2 months ago

Rising Sun is now shipping in physical form as well. (My copy is due to arrive on September 18.)

parkerc92 commented 1 month ago

Any updates on plans to add Rising Sun? I just purchased a physical copy from my local game store, can't wait to set up the tabs!

TistoH commented 1 month ago

I've gotten mine. I have added json for the all of the cards, card types (Omen, Prophecy, Shadow), and the set, including the en_us files. I am missing the "extra" from the rulebooks (for the cards and card types), plus some more checking.

I added "Rising Sun" to the bonuses. I used "Rs" for the text set symbol. I made crappy versions of prophecy.png and risinsun_set.png, and the latter may be broken for transparency?

My cygwin installation is not liking my attempts to update/install locally so that I can test (or use!) everything.

Unless somebody is further along, I can keep plugging away on "extra" text.

sumpfork commented 1 month ago

I've gotten mine. I have added json for the all of the cards, card types (Omen, Prophecy, Shadow), and the set, including the en_us files. I am missing the "extra" from the rulebooks (for the cards and card types), plus some more checking.

I added "Rising Sun" to the bonuses. I used "Rs" for the text set symbol. I made crappy versions of prophecy.png and risinsun_set.png, and the latter may be broken for transparency?

My cygwin installation is not liking my attempts to update/install locally so that I can test (or use!) everything.

Unless somebody is further along, I can keep plugging away on "extra" text.

Great, thanks. I suggest making a PR sooner rather than later with your work in progress so we can have a look, make fixes, etc. Can land it without the extras, too, and just add the extras later.

TistoH commented 1 month ago

Unfortunately, the problem with installing locally is even keeping me from committing (because of the commit hook) on my own machine. I have a patch file (made with "git diff --binary" so that the -- probably useless -- stand-in png files show up). Tisto_RisingSun.patch.gz

sumpfork commented 1 month ago

https://github.com/sumpfork/dominiontabs/pull/535 - please start working from that branch to avoid conflicts.

sumpfork commented 1 month ago

This is deployed to the online generator.