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 key that was removed at some point #22

Merged
merged 1 commit into from
Jul 13, 2024

Conversation

kihayu
Copy link
Contributor

@kihayu kihayu commented Jul 12, 2024

I noticed that in the NPC Character sheets the "Proficiency" on the top right was missing a key with a translation. After some digging inside the code and previous commits I noticed that a custom key is set because in German the value would be too long (typisch Deutsch). The key was already there once but got removed. So here it is again 🙂

Didn't test it but the key is still the same inside the code, so should be fine...

GIF

@kihayu
Copy link
Contributor Author

kihayu commented Jul 12, 2024

@mhilbrunner fyi

@mhilbrunner mhilbrunner merged commit 9e0cdd2 into mhilbrunner:master Jul 13, 2024
@mhilbrunner
Copy link
Owner

Typical Deutsch indeed. Thanks for the fix, the ping and also the digging in the first place :) Released version 3.2.1 with this change: https://github.com/mhilbrunner/foundryvtt-dnd5e-lang-de/releases/tag/3.2.1

@Szerina92
Copy link
Contributor

Looked through my stuff, was 100% my oopsie from my last commit.
I'm sorry, I will be more careful next time!

@mhilbrunner
Copy link
Owner

No worries, mistakes happen :) Thanks for contributing in the first place!

@kihayu kihayu deleted the bugfix/missing-translation branch July 16, 2024 21:15
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.

3 participants