-
Notifications
You must be signed in to change notification settings - Fork 13
Update for DnD5e 4.2.2 Final #38
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
base: master
Are you sure you want to change the base?
Conversation
Don't worry :) Git can be difficult to learn. It took me a long time to become even somewhat proficient, and that's only because I use it every day at work.
So most of the translations are still exactly the same as we discussed in the last PR? In that case I'll just skim through it again, since most of it should already be fine, or do you think I should read through it again in full? Also, @mhilbrunner, could you please check whether this PR can be merged without any problems? GitHub says there are still conflicts, but also that the changes could be cleanly merged. That sounds contradictory to me, but I don't have the permissions to look into it. |
Most are the same, I basically used my old stuff, I just changed some keys using the official wording, since the phb is out now. As soon as it's accepted I will start working on 4.3.x I synced my fork and used the latest file, don't know why there are those conflicts :( |
Alright, glad to hear this, that saves me a lot of time :) And don't worry too much about the conflicts, I'm quite sure this is only a misidentification if you just synced your fork. Which can happen occasionally. For some reason GitHub is not 100% sure it can merge the changes automatically, so it want's a human to have a look to avoid issues. As long as the changes can be merged without problems we can simply ignore that warning, but that is something the other Max has to take a look at since he's the only one with permissions to review the conflicts. In the unlikely case that there actually still are some, which I doubt, we'll also find a solution for that. :) |
Hey! Thanks again, both of you :) There are indeed some conflicts that prevent (auto) merging, I'll have a look. We really need to get this thing up on Weblate to get away from the Git workflow just for translation. I'm setting aside some time this weekend to try to start that. If you want to get in touch on Discord (I'm on the official Foundry Discord under the same name), maybe we should create a server there to discuss these things. |
I'm very sorry it took me so long, things have been very chaotic lately.
Also, for whatever reason, the following keys went back to their untranslated english form:
Other than that, well done! :) |
Thanks for your time! There are different "Trinkets".
I guess there were some problems with syncing the fork, those are some weird errors! |
Okay, I see. Thank you for the explanation! In that case, I don't think we should change it back. I just looked at the translation keys, and they both mention "trinket," so I thought it might have been an oversight. However, you obviously thought this through and intentionally chose two different translations. Since this is even closer to the official German books, let's keep it that way. Sounds good to me! :) Yeah, if I recall correctly, those were things I changed in my last 4.1 translation update. Especially the "Beidgändigkeit" error I clearly remember correcting. Most likely some error while syncing your fork. But whatever went wrong, at least those are pretty quick to rectify. |
It took a while...
Unfortunately, I haven't been able to learn how to do a “rebase.” And my skills haven't improved in other areas either! :)
I took the current data and merged it manually. I also made a few wording changes.
As always, I'm open to discussion.
After that, I would move on to 4.3.x (if necessary) and then 4.4.x to complete the 4.x series.