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

Translated the norwegian ifelse article. #54

Open
wants to merge 2 commits into
base: master
from

Conversation

@MathiasWP
Copy link

@MathiasWP MathiasWP commented Apr 26, 2020

I noticed that my prettier changed some small layout-structure. Is this a big deal?

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-no Apr 26, 2020
@CLAassistant
Copy link

@CLAassistant CLAassistant commented Apr 26, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


MathiasWP seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.


```js run no-beautify
let company = prompt('Which company created JavaScript?', '');
let selskap = prompt('Hvilket selskap lagde JavaScript?', '');
*!*
if (company == 'Netscape') {

This comment has been minimized.

@MathiasWP

MathiasWP Apr 26, 2020
Author

Company here should be "selskap", sorry for missing this...

This comment has been minimized.

@JonasJore

JonasJore Apr 27, 2020
Member

It is stated in the contributions guide that only the article is to be translated. Code examples should stay as is. PR looks good otherwise. I am a bit unsure about tasks though.
What do you think @iliakan?

This comment has been minimized.

@MathiasWP

MathiasWP Apr 27, 2020
Author

Oh okay, i guess that's a valid point. What's the reason as to why the code examples shouldn't be translated as well? However i will do it now!

This comment has been minimized.

@JonasJore

JonasJore Apr 29, 2020
Member

That is probably to keep the code working as expected across all of the different translations in this tutorial. This looks good to me now :) good job!

</script>

</body>
let verdi = prompt("Skriv inn et tall", 0);

This comment has been minimized.

@JonasJore

JonasJore Apr 27, 2020
Member

ref earlier comment in this PR

@javascript-translate-bot

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

MathiasWP
Copy link
Author

@MathiasWP MathiasWP left a comment

My prettier config has changed some of the layout structure. Is this okay?

@MathiasWP MathiasWP requested a review from JonasJore Apr 27, 2020
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

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