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

Add German translation #187

Merged
merged 29 commits into from
Jun 11, 2024
Merged

Add German translation #187

merged 29 commits into from
Jun 11, 2024

Conversation

Kurty00
Copy link

@Kurty00 Kurty00 commented Aug 11, 2023

No description provided.

@Noaaan Noaaan requested review from Noaaan and gliscowo August 11, 2023 20:52
@Noaaan Noaaan added the translations Related to the translations of the mod label Aug 12, 2023
Copy link
Owner

@Noaaan Noaaan left a comment

Choose a reason for hiding this comment

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

Thank you for taking time to translate Mythic Metals to German! I genuinely appreciate the time spent on this. Unfortunately I have multiple issues with this translation, please see the review comments.

Due to the state of the translation I will also mark this as a draft, as I do not believe it is ready to be merged.

@Noaaan Noaaan marked this pull request as draft August 28, 2023 11:41
@Kurty00 Kurty00 marked this pull request as ready for review September 8, 2023 12:43
@Kurty00 Kurty00 requested a review from Noaaan September 8, 2023 18:39
Copy link
Owner

@Noaaan Noaaan left a comment

Choose a reason for hiding this comment

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

Looks really nice now, awesome work!
Intending to have @gliscowo go over the translations regarding advancements if the idioms/references feel right. In the meantime I found some lines which seem untranslated, added some comments on those.

src/main/resources/assets/mythicmetals/lang/de_de.json Outdated Show resolved Hide resolved
src/main/resources/assets/mythicmetals/lang/de_de.json Outdated Show resolved Hide resolved
src/main/resources/assets/mythicmetals/lang/de_de.json Outdated Show resolved Hide resolved
src/main/resources/assets/mythicmetals/lang/de_de.json Outdated Show resolved Hide resolved
src/main/resources/assets/mythicmetals/lang/de_de.json Outdated Show resolved Hide resolved
src/main/resources/assets/mythicmetals/lang/de_de.json Outdated Show resolved Hide resolved
@Noaaan
Copy link
Owner

Noaaan commented Oct 22, 2023

Bumping this issue, as it seems to be stale

Mitigated mentioned issues
@Kurty00
Copy link
Author

Kurty00 commented Oct 28, 2023

Bumping this issue, as it seems to be stale

Sorry, didn't have much time lately...
I fixed some of the untranslated strings in the most recent commit now.

@Kurty00 Kurty00 requested a review from Noaaan October 28, 2023 20:57
@Noaaan
Copy link
Owner

Noaaan commented Oct 28, 2023

All good. Gonna try to get a proper review of this so its merged next week 👍

@Kurty00
Copy link
Author

Kurty00 commented Nov 16, 2023

Bumping this pr, as it seems to be stale.

@Noaaan
Copy link
Owner

Noaaan commented Jan 10, 2024

Sorry for this taking so long. Holidays set up all my german translation reviewers to be busy, including glisco. I have found someone with more MM knowledge to finally review the translation, and it should hopefully be done in the coming days/weeks. Going to post one of the review comments here, and add the rest of them once the review is complete.

@Noaaan Noaaan removed the request for review from gliscowo January 10, 2024 09:36
Copy link
Owner

@Noaaan Noaaan left a comment

Choose a reason for hiding this comment

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

Sorry for the long wait! I have finally received all the notes from the second party I wanted to review this translation.
I have split up the document they made into separate comments. Feel free to voice any thoughts/concerns surrounding them.

@Kurty00
Copy link
Author

Kurty00 commented Mar 30, 2024

Alright, I addressed most of the issues you/your second party noted. If I didn't, I should have provided comments.
Happy Easter!

@Kurty00 Kurty00 requested a review from Noaaan March 30, 2024 13:31
@Kurty00
Copy link
Author

Kurty00 commented Jun 8, 2024

I addressed the previously unresolved notes.

@Noaaan
Copy link
Owner

Noaaan commented Jun 8, 2024

Ah thank you! Just to let you know, I plan on having this merged for the last 1.20 release, followed by porting it to 1.20.6+.
A sidenote: some of the tag translations might change for the port, due to the major tag unification between NeoForge and Fabric. I would appreciate if you are available to look at it when the time comes

@Noaaan Noaaan merged commit 830e724 into Noaaan:1.20 Jun 11, 2024
1 check passed
@Kurty00
Copy link
Author

Kurty00 commented Jun 11, 2024

If you need me to look at some changes in the translation, feel free to contact me.

@Kurty00 Kurty00 deleted the Kurty00-language-de branch June 11, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translations Related to the translations of the mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants