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.

Sign up for GitHub

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

Jump to bottom

Remove {{ifmethod}} from pt-BR #7984

Merged
merged 1 commit into from
Aug 25, 2022
Merged

Remove {{ifmethod}} from pt-BR #7984

merged 1 commit into from
Aug 25, 2022

Conversation

SphinxKnight
Copy link
Member

@SphinxKnight SphinxKnight requested a review from a team as a code owner August 24, 2022 19:12
@SphinxKnight SphinxKnight requested review from josielrocha and removed request for a team August 24, 2022 19:12
@github-actions github-actions bot added the l10n-pt-br Issues related to Brazilian Portuguese label Aug 24, 2022
@SphinxKnight SphinxKnight added this to the Macros removal milestone Aug 24, 2022
@github-actions
Copy link
Contributor

Preview URLs

Flaws

URL: /pt-BR/docs/Web/API/Window/alert
Title: Window.alert
on GitHub
Flaw count: 4

  • macros:
    • This macro has been deprecated, and should be removed.
  • images:
    • External image URL
  • broken_links:
    • Can't resolve /en-US/docs/Chrome
  • translation_differences:
    • Differences in the important macros (1 in common of 3 possible)

URL: /pt-BR/docs/Web/API/Window/innerHeight
Title: Window.innerHeight
on GitHub
Flaw count: 11

  • macros:
    • Unknown macro todo
    • /pt-BR/docs/Web/API/Window/resizeBy does not exist but fell back to /en-US/docs/Web/API/Window/resizeBy
    • /pt-BR/docs/Web/API/Window/resizeTo does not exist but fell back to /en-US/docs/Web/API/Window/resizeTo
    • /pt-BR/docs/Web/API/Window/outerHeight does not exist but fell back to /en-US/docs/Web/API/Window/outerHeight
    • This macro has been deprecated, and should be removed.
    • and 4 more flaws omitted
  • images:
    • External image URL
  • translation_differences:
    • Differences in the important macros (1 in common of 5 possible)

External URLs

URL: /pt-BR/docs/Web/API/Window/alert
Title: Window.alert
on GitHub

No new external URLs

URL: /pt-BR/docs/Web/API/Window/innerHeight
Title: Window.innerHeight
on GitHub

No new external URLs
josielrocha
josielrocha approved these changes Aug 25, 2022
View reviewed changes
Copy link
Collaborator

@josielrocha josielrocha left a comment

Choose a reason for hiding this comment

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

LGTM ;)
@josielrocha josielrocha merged commit c77cb3e into mdn:main Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-pt-br Issues related to Brazilian Portuguese
2 participants
@SphinxKnight @josielrocha