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 translation added #243

Merged
merged 4 commits into from
Jun 16, 2022
Merged

Conversation

drct
Copy link
Contributor

@drct drct commented Jun 8, 2022

Added de.lproj folder containing Localizable.strings with German translation. No changes to Xcode's project file. First translation and github pull request for me, so I hope everything's alright. Cheers.

@drct drct requested a review from a team as a code owner June 8, 2022 12:53
@MattKiazyk
Copy link
Contributor

Hey @drct looks like your missing some changes to the XcodeProj to include it? I was going to take your PR for German because you create it first and the dictionary is the exact same as #245. If you're ok with it, I can take the other one instead, otherwise, could you fix it up to include those changes. Thanks

@drct
Copy link
Contributor Author

drct commented Jun 13, 2022

@MattKiazyk Thanks. I have updated the branch to include also the changes to the XcodeProj file. Does this work for you?

@MattKiazyk MattKiazyk added the enhancement New feature or request label Jun 13, 2022
Copy link
Contributor

@MattKiazyk MattKiazyk left a comment

Choose a reason for hiding this comment

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

@drct Just one small change needed. Thanks!

Co-authored-by: Matt Kiazyk <mattkiazyk@gmail.com>
@drct
Copy link
Contributor Author

drct commented Jun 14, 2022

@MattKiazyk Created a commit with your suggestions.

@MattKiazyk MattKiazyk merged commit 8831d5f into XcodesOrg:main Jun 16, 2022
@MattKiazyk
Copy link
Contributor

Thanks for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants