GV

GeezerVison

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

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Property flags and descriptors by kenliten · Pull Request #183 · javascript-tutorial/es.javascript.info

Conversation

@kenliten

@kenliten kenliten commented May 12, 2020

Copy link
Copy Markdown
Contributor

No description provided.

Sorry, something went wrong.

Uh oh!

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

All reactions
Otoniel Reyes Galay
Property flags and descriptors
6075c98
@javascript-translate-bot javascript-translate-bot added the review needed label May 12, 2020
@javascript-translate-bot
javascript-translate-bot requested a review from a team May 12, 2020 03:16
@CLAassistant

CLAassistant commented May 12, 2020 •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown

CLA assistant check
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 iliakan mentioned this pull request May 12, 2020
Spanish Translation Progress #17
Open
@vplentinax
Merge branch 'master' into master
d8b3d1b
EzequielCaste
EzequielCaste requested changes Jun 24, 2020
View reviewed changes

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor

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.

Hay problemas con el número de las líneas a partir de línea 127. Verificar con artículo original EN

Alguna sugerencia?

Sorry, something went wrong.

Uh oh!

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

All reactions
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

@javascript-translate-bot javascript-translate-bot removed the review needed label Jun 24, 2020
@javascript-translate-bot

javascript-translate-bot commented Jun 24, 2020

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.

@javascript-translate-bot javascript-translate-bot added the changes requested label Jun 24, 2020
vplentinax and others added 3 commits June 25, 2020 14:52
@vplentinax @EzequielCaste
Update 1-js/07-object-properties/01-property-descriptors/article.md
eed1c43
Co-authored-by: Ezequiel Castellanos <[email protected]>
@vplentinax @EzequielCaste
Update 1-js/07-object-properties/01-property-descriptors/article.md
531b243
Co-authored-by: Ezequiel Castellanos <[email protected]>
@vplentinax @EzequielCaste
Update 1-js/07-object-properties/01-property-descriptors/article.md
52788c2
Co-authored-by: Ezequiel Castellanos <[email protected]>
@vplentinax

vplentinax commented Jun 25, 2020

Copy link
Copy Markdown
Contributor

/done

All reactions

Sorry, something went wrong.

Uh oh!

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

vplentinax
vplentinax previously requested changes Jun 25, 2020
View reviewed changes

@vplentinax vplentinax left a comment

Copy link
Copy Markdown
Contributor

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.

Le faltaron partes por traducir. Las estructuras coinciden EN vs ES

Sorry, something went wrong.

Uh oh!

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

All reactions
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

@javascript-translate-bot

javascript-translate-bot commented Jun 25, 2020

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.

vplentinax added 2 commits June 30, 2020 13:26
@vplentinax
Update 1-js/07-object-properties/01-property-descriptors/article.md
5eea71f
@vplentinax
Update 1-js/07-object-properties/01-property-descriptors/article.md
befa1fe
@vplentinax

vplentinax commented Jun 30, 2020

Copy link
Copy Markdown
Contributor

/done

All reactions

Sorry, something went wrong.

Uh oh!

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

@vplentinax

vplentinax commented Jun 30, 2020

Copy link
Copy Markdown
Contributor

Volver a revisar

All reactions

Sorry, something went wrong.

Uh oh!

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

@vplentinax vplentinax added the review needed label Jun 30, 2020
joaquinelio added 2 commits June 30, 2020 20:05
@joaquinelio
Update article.md
840c9ae
añadido triple grave linea 127
@joaquinelio
Update article.md
654c618
Editado smart header y contenido  251 - 254
joaquinelio
joaquinelio approved these changes Jun 30, 2020
View reviewed changes

@joaquinelio joaquinelio 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.

Edité el archivo directamente, dejo registro:
127 agregado formato gravegravegrave,
no se puede hacer suggest porque el triple tilde lo cierra y borraria la linea
251-254 ??? el smart header es completamente distinto...
dejo en coment el original y también edito directamente por la misma razon del triple tilde.

Sorry, something went wrong.

Uh oh!

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

All reactions
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Outdated
Notable exception: a value of non-configurable, but writable property can be changed.

The idea of `configurable: false` is to prevent changes to property flags and its deletion, not changes to its value.
```smart header="Los errores aparecen solo en uso estricto"

@joaquinelio joaquinelio Jun 30, 2020

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.

Original 251-252
___smart header="Los errores aparecen solo en uso estricto"
En el modo no estricto, no aparecen errores al escribir en propiedades de solo lectura y semejantes. Pero la operación no será ejecutada. Las acciones viola-identificadores son silenciadas e ignoradas en modo no estricto.

Sorry, something went wrong.

Uh oh!

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

All reactions
@javascript-translate-bot javascript-translate-bot added needs +1 and removed changes requested review needed labels Jun 30, 2020
@joaquinelio
Update article.md
b2c4ded
fix a la edición de 254
EzequielCaste
EzequielCaste approved these changes Jul 1, 2020
View reviewed changes

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor

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.

todo bien salvo algunas correcciones

Sorry, something went wrong.

Uh oh!

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

All reactions
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Resolved

Uh oh!

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

@EzequielCaste
Update 1-js/07-object-properties/01-property-descriptors/article.md
c1db644
EzequielCaste added 2 commits July 1, 2020 10:52
@EzequielCaste
Update 1-js/07-object-properties/01-property-descriptors/article.md
ec8915b
@EzequielCaste
Update 1-js/07-object-properties/01-property-descriptors/article.md
74c5820
@EzequielCaste
EzequielCaste dismissed vplentinax’s stale review July 1, 2020 13:53

ya fue revisado

@EzequielCaste
EzequielCaste merged commit 02780d9 into javascript-tutorial:master Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joaquinelio joaquinelio joaquinelio approved these changes
@EzequielCaste EzequielCaste EzequielCaste approved these changes
@vplentinax vplentinax vplentinax left review comments

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

@kenliten @CLAassistant @javascript-translate-bot @vplentinax @joaquinelio @EzequielCaste
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.