GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — Conditional operators: if, '?' by Giorgiosaud · Pull Request #95 · javascript-tutorial/es.javascript.info

Conversation

Giorgiosaud commented Sep 8, 2019

Copy link

Copy Markdown

Contributor

Conditional operators: if, '?'

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

Giorgiosaud added 2 commits

September 7, 2019 20:23


          Conditional operators: if, '?'

f46d8ff


          Merge pull request #1 from Giorgiosaud/Conditional-operators-if

dd2a4c9

Conditional operators: if, '?'

javascript-translate-bot added the review needed label

Sep 8, 2019

javascript-translate-bot requested a review from a team

September 8, 2019 00:26

CLAassistant commented Sep 8, 2019 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link

Copy Markdown

All committers have signed the CLA.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

iliakan mentioned this pull request

Sep 8, 2019

Spanish Translation Progress #17

Open

tscandalitta reviewed

Sep 8, 2019

View reviewed changes

Comment thread

1-js/02-first-steps/10-ifelse/1-if-zero-string/solution.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

tscandalitta reviewed

Sep 8, 2019

View reviewed changes

Comment thread

1-js/02-first-steps/10-ifelse/1-if-zero-string/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

tscandalitta added the changes requested label

Sep 8, 2019

Giorgiosaud and others added 5 commits

September 7, 2019 22:54


          functions translations

486eeaa


          Functions translated

5f80167


          Merge pull request #2 from Giorgiosaud:Functions

7ba737b

Functions translated


          Update solution.md

8110c85


          Update task.md

0a1aab1

Giorgiosaud requested review from tscandalitta and removed request for a team

September 8, 2019 03:20

tscandalitta removed the review needed label

Sep 8, 2019

lizzie136 requested changes

Sep 10, 2019

View reviewed changes

lizzie136 left a comment

Copy link

Copy Markdown

Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Aún no he terminado este review, pero te dejo los comentarios que ya tengo contemplados. Gracias @Giorgiosaud!!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

Comment thread

1-js/02-first-steps/10-ifelse/2-check-standard/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

1-js/02-first-steps/10-ifelse/5-rewrite-if-question/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

1-js/02-first-steps/10-ifelse/6-rewrite-if-else-question/task.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

1-js/02-first-steps/10-ifelse/article.md Outdated

		# Operadores condicionales: if, '?'

		Sometimes, we need to perform different actions based on different conditions.
		Algunas Veces, necesitamos ejecutar diferentes acciones basadas en diferentes condiciones.

lizzie136 Sep 10, 2019

Copy link

Copy Markdown

Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Algunas veces

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

Comment thread

1-js/02-first-steps/10-ifelse/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

1-js/02-first-steps/10-ifelse/article.md Outdated

		## Conversion Booleana

		The `if (…)` statement evaluates the expression in its parentheses and converts the result to a boolean.
		La sentencia `if (…)` evalua la exprecion dentro de sus parentesis y convierte el resultado en booleano.

lizzie136 Sep 10, 2019

Copy link

Copy Markdown

Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

expresión, y paréntesis

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

Comment thread

1-js/02-first-steps/10-ifelse/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread

1-js/02-first-steps/10-ifelse/article.md Outdated

		## La clausula "else"

		The `if` statement may contain an optional "else" block. It executes when the condition is false.
		La sentencia `if` quizás contenga un bloque "de lo contrario" opcional. Este se ejecutará cuando la condición sea falsa.

lizzie136 Sep 10, 2019

Copy link

Copy Markdown

Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

creo que en este caso "else" no debería ir traducido, o al menos si vas a poner la traducción podría ir como que se traduce o lee como "de lo contrario"

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

Comment thread

1-js/02-first-steps/10-ifelse/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

javascript-translate-bot commented Sep 10, 2019

Copy link

Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Giorgiosaud added 4 commits

September 10, 2019 21:15


          Update task.md

776f436


          Update task.md

d1ccb8c


          fix acentos y otros

69a9ec5

gracias @lizzie136


          Update task.md

d8d25cf

Giorgiosaud commented Sep 11, 2019

Copy link

Copy Markdown

Contributor Author

/done

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

javascript-translate-bot added review needed and removed changes requested labels

Sep 11, 2019

javascript-translate-bot requested a review from a team

September 11, 2019 00:21

Giorgiosaud requested review from lizzie136 and removed request for a team

September 11, 2019 00:21

lizzie136 approved these changes

Oct 4, 2019

View reviewed changes

lizzie136 left a comment

Copy link

Copy Markdown

Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Gracias, en adelante, hay lugares donde dice parametros sin tilde, que debería ser parámetros.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

javascript-translate-bot added needs +1 and removed review needed labels

Oct 4, 2019

vplentinax mentioned this pull request

Jun 5, 2020

Functions #230

Merged

vplentinax closed this

Jun 5, 2020

vplentinax reopened this

Jun 5, 2020

vplentinax mentioned this pull request

Jun 5, 2020

Conditional operators: if, '?' #231

Merged

vplentinax closed this

Jun 5, 2020

Giorgiosaud deleted the master branch

October 30, 2020 12:45

Reviewers

lizzie136

lizzie136 approved these changes

tscandalitta

Awaiting requested review from tscandalitta

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

needs +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

6 participants

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.

GeezerVison

Conversation

Giorgiosaud commented Sep 8, 2019

Uh oh!

CLAassistant commented Sep 8, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lizzie136 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

lizzie136 Sep 10, 2019

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lizzie136 Sep 10, 2019

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lizzie136 Sep 10, 2019

Choose a reason for hiding this comment

Uh oh!

Uh oh!

javascript-translate-bot commented Sep 10, 2019

Uh oh!

Giorgiosaud commented Sep 11, 2019

Uh oh!

lizzie136 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

CLAassistant commented Sep 8, 2019 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.