Skip to content

chore(macros): remove page macro#11356

Closed
fiji-flo wants to merge 4 commits intomdn:mainfrom
fiji-flo:remove-page-macro
Closed

chore(macros): remove page macro#11356
fiji-flo wants to merge 4 commits intomdn:mainfrom
fiji-flo:remove-page-macro

Conversation

@fiji-flo
Copy link
Copy Markdown
Contributor

Summary

the page macro has been deprecated and is broken anyway

the page macro has been deprecated and is broken anyway
@github-actions github-actions Bot added the macros tracking issues related to kumascript macros label Jun 24, 2024
@fiji-flo fiji-flo marked this pull request as ready for review June 26, 2024 15:06
@fiji-flo fiji-flo requested a review from a team as a code owner June 26, 2024 15:06
@fiji-flo
Copy link
Copy Markdown
Contributor Author

Running

rg "\{\{\s*page\(" -i

on translated content shows the only uses in pt-BR. And there it's broken already (see https://developer.mozilla.org/pt-BR/docs/Web/API/File#propriedades)


// Throw a MacroDeprecatedError flaw
// Can be removed when its usage translated-content is down to 0
mdn.deprecated();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@fiji-flo Since this macro is still referenced in pt-BR, can we please keep a stub with mdn.deprecated() so that localizers notice?

@github-actions github-actions Bot added the idle label Aug 3, 2024
@github-actions github-actions Bot removed the idle label Aug 16, 2024
@github-actions github-actions Bot added the idle label Sep 16, 2024
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions Bot added merge conflicts 🚧 Please rebase onto or merge the latest main. and removed idle labels Oct 29, 2024
@github-actions github-actions Bot added the idle label Nov 29, 2024
@github-actions github-actions Bot removed the idle label Apr 24, 2025
@github-actions github-actions Bot added the idle label May 24, 2025
@caugner caugner closed this Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

idle macros tracking issues related to kumascript macros merge conflicts 🚧 Please rebase onto or merge the latest main.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants