Revising card names and descriptions for en & de #550
Replies: 2 comments 3 replies
-
JSON keys are automatically sorted by the pre commit hooks, and I'd like to keep it that way exactly so that I don't have to check each contribution for not messing up some otherwise unenforced sorting. |
Beta Was this translation helpful? Give feedback.
-
What Nick said, but also let me be clear: I'm afraid it's just not true that you need zero technical experience to change the language files. They are in a specific format that can break the script if you don't follow it. You should be at minimum be able to package the files, run the linters and tests to make a working pull request. Anything else just creates a huge amount of work on my side to fix up people's submissions - I don't have time for that, sadly. We can ask for help with submission other than working PRs, but that'll rely on help from the community unless they require only small fixes. |
Beta Was this translation helpful? Give feedback.
-
Hey, I'd like to revise the whole jsons for cards. Given I only can speak english and german, I'd stick with those. There are a few more questions before I begin.
For current and future ease, I'd re-organize the whole file by expansion->alphabet instead of global alphabet. It really is a pain to look up all cards the current way. Especially since the integrity ofd the old system was broken by another user who started a new alphabet at the bottom. Also, using https://wiki.dominionstrategy.com/index.php/List_of_cards enables users to quickly update the file, at least for english. For other languages, it is easier to work down the file with the website.
I don't think I have the expertise to edit all extras. Descriptions are easily copied. I could copy the extras from the rulebooks, but those could be outdated by erratas. I think either way, that they have no priority.
How to deal with the renaming of Harem? Name it "Harem / Farm"?
Also, this has nothing to do with the cards description, but cards_bd has Tide Pools wrongly priced (3 instead of 4). I'd upload this file aswell, so just pulling it would be needed.
Beta Was this translation helpful? Give feedback.
All reactions