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

Remove deprecated DiscussionList macro from es #9608

Merged
merged 2 commits into from
Oct 31, 2022

Conversation

Graywolf9
Copy link
Contributor

Description

Remove deprecated DiscussionList macro from es

Process: remove the macro and replace with the rendered content

Motivation

The chore of deprecated macros removal

Additional details

Related issues and pull requests

Relates to https://github.com/orgs/mdn/discussions/263

@Graywolf9 Graywolf9 added the l10n-es Issues related to Spanish content. label Oct 31, 2022
@Graywolf9 Graywolf9 requested a review from a team as a code owner October 31, 2022 18:37
@Graywolf9 Graywolf9 requested review from JuanVqz and removed request for a team October 31, 2022 18:37
@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2022

Preview URLs (6 pages)
Flaws (57)

Note! 1 document with no flaws that don't need to be listed. 🎉

URL: /es/docs/Web/MathML
Title: MathML
Flaw count: 7

  • macros:
    • Calling the Compat macro with any arguments is deprecated; instead use the 'browser-compat' front-matter key.
  • broken_links:
    • Can't resolve /es/docs/Introducción_a_XML
    • Can't resolve /es/docs/Web/MathML/Elemento
    • Can use the English (en-US) link as a fallback
    • Can't resolve /es/docs/tag/MathML
    • Is currently http:// but can become https://
  • translation_differences:
    • Differences in the important macros (0 in common of 2 possible)

URL: /es/docs/Web/SVG
Title: SVG
Flaw count: 25

  • macros:
    • /es/docs/Glossary/SMIL does not exist
  • broken_links:
    • Can't resolve /es/docs/SVG/Tutorial
    • Can use the English (en-US) link as a fallback
    • Can't resolve /es/docs/web/HTML
    • Can't resolve /es/docs/CSS
    • Can use the English (en-US) link as a fallback
    • and 18 more flaws omitted
  • translation_differences:
    • Differences in the important macros (0 in common of 1 possible)

URL: /es/docs/Web/Guide/AJAX
Title: AJAX
Flaw count: 18

  • broken_links:
    • Can't resolve /es/HTML
    • Can't resolve /es/XHTML
    • Can't resolve /es/CSS
    • Can't resolve /es/JavaScript
    • Can't resolve /es/DOM
    • and 13 more flaws omitted

URL: /es/docs/Web/Accessibility/Community
Title: Comunidad
Flaw count: 1

  • broken_links:
    • Is currently http:// but can become https://

URL: /es/docs/Web/XSLT
Title: XSLT
Flaw count: 6

  • broken_links:
    • Can't resolve /es/Transformando_XML_con_XSLT
    • Can't resolve /es/XSLT/Elementos
    • Is currently http:// but can become https://
    • Can't resolve /es/XML
    • Can't resolve /es/XPath
    • and 1 more flaws omitted
External URLs (21)

URL: /es/docs/Web/MathML
Title: MathML


URL: /es/docs/Web/SVG
Title: SVG


URL: /es/docs/Web/Guide/AJAX
Title: AJAX


URL: /es/docs/Web/Guide/AJAX/Community
Title: Comunidad


URL: /es/docs/Web/Accessibility/Community
Title: Comunidad


URL: /es/docs/Web/XSLT
Title: XSLT

(this comment was updated 2022-10-31 20:23:07.003155)

Copy link
Member

@JuanVqz JuanVqz left a comment

Choose a reason for hiding this comment

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

Gracias por contribuir y ser parte activa de nuestra comunidad en español :)
(no se que haríamos sin ti! 💯 )

Deje un pequeño comentario

files/es/web/svg/index.md Outdated Show resolved Hide resolved
Co-authored-by: Juan Vásquez <javasgon@gmail.com>
@Graywolf9
Copy link
Contributor Author

Muchas gracias por la observación @JuanVqz

@JuanVqz JuanVqz merged commit 9629fc5 into mdn:main Oct 31, 2022
@Graywolf9 Graywolf9 deleted the es-remove-DiscussionList-macro branch July 5, 2023 22:08
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.

2 participants