Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

German translations for "Rising Sun" #547

Merged
merged 4 commits into from
Oct 30, 2024
Merged

Conversation

s3inlc
Copy link
Contributor

@s3inlc s3inlc commented Oct 30, 2024

Hey there :)

I added the German translations for all the cards of the "Rising Sun" expansion, which hopefully others are interested to get as well.

Please let me know if there is anything else I need to add or change in order to make the PR correctly.

@sumpfork sumpfork self-requested a review October 30, 2024 20:48
Copy link
Contributor

It seems compiling the card DB on this PR produced changes.
Ideally, please run "doit update-language" yourself on this PR and check in the results.
Otherwise, PR #548 has the necessary (but unreviewed) changes.

@sumpfork
Copy link
Owner

Hey there :)

I added the German translations for all the cards of the "Rising Sun" expansion, which hopefully others are interested to get as well.

Please let me know if there is anything else I need to add or change in order to make the PR correctly.

Vielen Dank!

Could you re-compile the card db one more time, I think it just needs your small fix (second commit in this PR) integrated perhaps?

@s3inlc
Copy link
Contributor Author

s3inlc commented Oct 30, 2024

Ah indeed, I may have forgot that after the small change. I re-ran the update_language command again and pushed.

Copy link
Owner

@sumpfork sumpfork left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich habe nicht alles durchgelesen, aber zufällig doch ein fehlendes "t" bemerkt.

card_db_src/de/sets_de.json Outdated Show resolved Hide resolved
@sumpfork sumpfork self-requested a review October 30, 2024 22:32
@sumpfork sumpfork merged commit 1c61f77 into sumpfork:master Oct 30, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants