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

Update index.md #13795

Closed
wants to merge 1 commit into from
Closed

Update index.md #13795

wants to merge 1 commit into from

Conversation

evazquezb
Copy link
Contributor

@evazquezb evazquezb commented Jun 17, 2023

without-permissions-getUserMedia-return-NotAllowedError

Description

Se puede tener problemas a la hora de manejar errores ejecutando el metodo getUserMedia. En la traduccion dice, que el metodo devuelve un "PermissionDeniedError" esto implica que ese es el nombre del error pero la verdad que el nombre es "NotAllowedError" y el mensaje es "Permission denied". Esto me causo conflictos ya que busque en la documentacion y no podia manejar el error ni mediante la propiedad "name" ni mediante la propiedad "message" debido a que PermissionDeniedError no lo obtenia en ningun lado.

Motivation

Additional details

Related issues and pull requests

 without-permissions-getUserMedia-return-NotAllowedError
@evazquezb evazquezb requested a review from a team as a code owner June 17, 2023 00:29
@evazquezb evazquezb requested review from Graywolf9 and removed request for a team June 17, 2023 00:29
@github-actions github-actions bot added the l10n-es Issues related to Spanish content. label Jun 17, 2023
@vickvasquez
Copy link
Contributor

Ya existe un PR que resuelve el mismo problema #13794

@evazquezb evazquezb closed this Jun 17, 2023
@evazquezb evazquezb deleted the patch-3 branch June 17, 2023 01:06
@evazquezb
Copy link
Contributor Author

evazquezb commented Jun 17, 2023

Ya existe un PR que resuelve el mismo problema #13794

Bueno, de hecho lo crearon debido a un PR de una rama que borre y resuelve el problema a medias, y de hechola rama de este PR la borre tambien, por lo mismo,. Perdon pero es que soy nuevo en esto.

Este PR resuelve el malentendido a medias, puesto que tambien hay otra parte que conserva el error, en un codigo de ejemplo, al atrapar el error valida que sea de nombre "PermissionDeniedError".

Pero cree otra rama y un PR #13797 donde modifico ambas partes, la parte modificada en este PR y la que contiene el codigo de ejemplo.

@Graywolf9
Copy link
Contributor

Hola! @evazquezb muchas gracias por tu apoyo, veo que el nuevo PR que mencionas también lo has cerrado, aprovecho para comentarte que para solución de errores puntuales lo mejor es enviar PRs separados por cada artículo, en este caso podemos dejar el PR #13794 para ese artículo y si gustas crear otro PR solo con el otro artículo para poder revisar cada caso separado y uno no atrase al otro.

Nuevamente muchas gracias por todo @evazquezb y estamos para ayudar en lo que necesites.

También gracias @vickvasquez por la revisión

@evazquezb
Copy link
Contributor Author

evazquezb commented Jun 19, 2023

@Graywolf9
Ok, gracias.
En un código de ejemplo observé que se usa también "PermissionDeniedError" como nombre de error
Lo cambié e hice un PR #13838
En la versión en inglés ya se hizo el cambio correspondiente. [#27415]: mdn/content#27415
Ojalá acá en la traducción pueda darse también el cambio.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-es Issues related to Spanish content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants