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

Macro - Meta - Remove transclusion {{EmbedLiveSample}} calls from es #6509

Closed
30 tasks done
JuanVqz opened this issue Jun 26, 2022 · 1 comment · Fixed by #9902
Closed
30 tasks done

Macro - Meta - Remove transclusion {{EmbedLiveSample}} calls from es #6509

JuanVqz opened this issue Jun 26, 2022 · 1 comment · Fixed by #9902
Labels
documentation Improvements or additions to documentation l10n-es Issues related to Spanish content.

Comments

@JuanVqz
Copy link
Member

JuanVqz commented Jun 26, 2022

Hola contribuyentes de MDN en español. Estamos mejorando la macro EmbedLiveSample y tenemos que revisar los documentos en la lista para hacerlos coincidir con la versión en inglés.

Pueden haber diferentes formas de resolver los casos en esta lista, así que presta mucha atención y si tienes dudas pregunta en este mismo issue, gracias 🚀

Hice el primer punto como ejemplo y explicaré lo que hice en un comentario a continuación.

Otras referencias: #5717

Related to #5716

To-do

  • Update content with English versión
  • Convert to md
    Con los dos anteriores automáticamente queda el siguiente
  • Reformatear la macro EmbedLiveSample

Example
From
{{ EmbedLiveSample('Rect_code', '700', '300', '', 'Games/Techniques/2D_collision_detection') }}
To (in some cases needs to write the code
{{ EmbedLiveSample('', '700', '300', '') }}

@Graywolf9
Copy link
Contributor

Hola! Corrí el comando provisto en mdn/content#15872

rg "EmbedLiveSample *\([^,]*,[^,]*,[^,]*,[^,]*, *'[^'][^']*'" -i

Y conseguí el número de archivos detectados en #5716

Revisé con los tuyos y las diferencias solo eran por que había más de una macro en algunos archivos.

Acomodo y creo algunos issue con la línea de cada uno.

Muchas gracias por la colaboración!

Captura de Pantalla 2022-07-08 a la(s) 11 14 42

@Jalkhov Jalkhov added the documentation Improvements or additions to documentation label Jul 18, 2022
@queengooborg queengooborg added this to the Macros removal milestone Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation l10n-es Issues related to Spanish content.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants