dominiontabs icon indicating copy to clipboard operation
dominiontabs copied to clipboard

feat: updated sets and translations for bigbox 2 german edition

Open despian opened this issue 4 years ago • 3 comments

A couple of cards in the German Bigbox v2 have slightly different translations.

I added 2 new sets (promo-bigbox2-de and guilds-bigbox2-de) to support the different names of the cards:

"Carcassonne" -> "Befestigtes Dorf" "Wahrsager" -> "Hellseherin"

despian avatar Mar 21 '20 14:03 despian

Sorry, missed that part of the readme. Should be good now.

Though the size of this diff just for these small changes makes me cry a little 😅

despian avatar Mar 23 '20 12:03 despian

I don't know why travis is not reporting the outcomes of tests correctly to github anymore, but see https://travis-ci.org/github/sumpfork/dominiontabs/builds/665841935

sumpfork avatar Mar 23 '20 18:03 sumpfork

Yeah, I see. I wrongly assumed the pre-commit hook was running the tests and that they had passed.

I've added these new sets to the failing test. Should be passing now. (Though redefining part of the card_db in a test seems a little odd to me, not very DRY...)

I'm not really happy with the number of changes required here for this seemingly simple thing. I'm sure there are better ways of achieving this but would probably require changing the underlying structure of the generator a bit. Which I'm not really up for tbh.

I accomplished what I needed anyhow and this would help others trying to use it for the same bigbiox version I have. That said, I understand if you don't want to merge this.

despian avatar Mar 27 '20 13:03 despian

Replaced by #440

sumpfork avatar Jan 10 '23 05:01 sumpfork