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 of code editors #5

Closed
wants to merge 16 commits into from

Conversation

@higoka
Copy link

@higoka higoka commented Jan 14, 2020

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-de Jan 14, 2020
@claassistantio
Copy link

@claassistantio claassistantio commented Jan 14, 2020

CLA assistant check
All committers have signed the CLA.

higoka and others added 7 commits Jan 15, 2020
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Copy link
Member

@1H0 1H0 left a comment

I think we should agree that we all translate the article in the du-form. So that we are consistent through the articles.

1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@1H0
Copy link
Member

@1H0 1H0 commented Jan 20, 2020

Just saw that the topic was not reserved by you and that @MichFriee reserved and translated the page in another PR. Please make sure to always check first, if the article isn't already taken and to reserve it first...

Ok when I close this PR and you take another article?

higoka and others added 5 commits Jan 20, 2020
Co-Authored-By: Lukas <37246258+1H0@users.noreply.github.com>
Co-Authored-By: Lukas <37246258+1H0@users.noreply.github.com>
Co-Authored-By: Lukas <37246258+1H0@users.noreply.github.com>
Co-Authored-By: Lukas <37246258+1H0@users.noreply.github.com>
Co-Authored-By: Lukas <37246258+1H0@users.noreply.github.com>
@higoka
Copy link
Author

@higoka higoka commented Jan 20, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-de Jan 20, 2020
1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
1-js/01-getting-started/3-code-editors/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

higoka and others added 3 commits Jan 21, 2020
Co-Authored-By: Gabriel Jörg <jergo3020@gmail.com>
Co-Authored-By: Gabriel Jörg <jergo3020@gmail.com>
Co-Authored-By: Gabriel Jörg <jergo3020@gmail.com>
@higoka
Copy link
Author

@higoka higoka commented Jan 21, 2020

/done

@MichFrie
Copy link

@MichFrie MichFrie commented Jan 21, 2020

Hi 1HO, sorry for the confusion, please feel free to close my commit and use the other translation if that's helpful

@1H0
Copy link
Member

@1H0 1H0 commented Jan 24, 2020

Hi @MichFrie!

You did everything right! You reserved the topic first, so it's fine! Just make sure to implement the changes requesten on your PR. Then we can merge your one!

I think i have to close this one now... Since it was translated twice...

@1H0
Copy link
Member

@1H0 1H0 commented Jan 24, 2020

Duplicated translation... @MichFrie reserved the Topic properly, because of that we'll merge his one when the requested changes are implemented.

Sorry @higoka ... 😞 Make sure to properly claim the topics in the future!

@1H0 1H0 closed this Jan 24, 2020
javascript-translate-bot pushed a commit that referenced this pull request Mar 24, 2020
Updated Repo
SirCodewright added a commit to SirCodewright/de.javascript.info that referenced this pull request Apr 5, 2020
Co-Authored-By: Christian Hegedüs <christianhegedues@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants
You can’t perform that action at this time.