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 WindowOrWorkerGlobalScope #5617

Open
8 of 9 tasks
queengooborg opened this issue May 13, 2022 · 8 comments
Open
8 of 9 tasks

Remove WindowOrWorkerGlobalScope #5617

queengooborg opened this issue May 13, 2022 · 8 comments
Labels
l10n-pt-br Issues related to Brazilian Portuguese

Comments

@queengooborg
Copy link
Collaborator

queengooborg commented May 13, 2022

As a follow-up to mdn/content#8796, the translated versions of the WindowOrWorkerGlobalScope page should be removed, and all of the pages for its subfeatures (which have been moved to the top level, but still have WindowOrWorkerGlobalScope.[member] as their title) should also be updated.

Affected locales:

Steps for each locale:

  • Remove the page for Web/API/WindowOrWorkerGlobalScope
  • Search for pages with a title that starts with WindowOrWorkerGlobalScope. -- for each page:
    • Move the page a folder up using yarn content move Web/API/WindowOrWorkerGlobalScope/<prop> Web/API/<prop> <locale> in mdn-content
    • Update the title to remove mention (AKA, WindowOrWorkerGlobalScope.caches becomes caches)
    • Update the translation_of key to remove mention (AKA, Web/API/WindowOrWorkerGlobalScope/caches becomes Web/API/caches)
    • Update browser-compat key to remove mention (AKA, api.WindowOrWorkerGlobalScope.caches becomes api.caches)
      • If there is no browser-compat key, but there is a {{Compat}} macro, update its value instead
      • If neither a browser-compat key or {{Compat}} macro call exist, ignore this step
  • Remove all {{domxref}} references to WindowOrWorkerGlobalScope (the regex domxref[ ]*\([ ]*["'][ ]*WindowOrWorkerGlobalScope may help find these instances)
@queengooborg queengooborg added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label May 13, 2022
@yin1999 yin1999 added l10n-ja Issues related to Japanese content. l10n-fr Issues related to French content. l10n-zh Issues related to Chinese content. l10n-es Issues related to Spanish content. l10n-ru Issues related to Russian content. l10n-ko Issues related to Korean content. l10n-pt-br Issues related to Brazilian Portuguese l10n-de (retired) Issues related to German content. l10n-pl (retired) Issues related to Polish content. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels May 27, 2022
SphinxKnight added a commit to SphinxKnight/translated-content that referenced this issue Jun 13, 2022
@SphinxKnight
Copy link
Member

@queengooborg regarding the original_slug, I'm not sure its edition is needed here. To me this is automatically filled by yari when moving documents.

@SphinxKnight
Copy link
Member

Regarding the last step: domxref[ ]*\([ ]*["'][ ]*WindowOrWorkerGlobalScope may be helpful to help detect occurrences.

@queengooborg
Copy link
Collaborator Author

Thanks @SphinxKnight! I've updated the instructions accordingly!

@yin1999 yin1999 removed the l10n-zh Issues related to Chinese content. label Jun 20, 2022
@cw118 cw118 closed this as completed in 2d70c55 Jun 21, 2022
@cw118
Copy link
Member

cw118 commented Jun 21, 2022

Done for fr, I didn't mean to close this :/

@cw118 cw118 reopened this Jun 21, 2022
@cw118 cw118 removed the l10n-fr Issues related to French content. label Jun 21, 2022
@yin1999 yin1999 removed l10n-de (retired) Issues related to German content. l10n-pl (retired) Issues related to Polish content. labels Aug 30, 2022
@mfuji09
Copy link
Collaborator

mfuji09 commented Jan 13, 2023

done for ja locale.

@mfuji09 mfuji09 removed the l10n-ja Issues related to Japanese content. label Jan 13, 2023
@Graywolf9
Copy link
Contributor

Done for es , thank you!

@Graywolf9 Graywolf9 removed the l10n-es Issues related to Spanish content. label Feb 9, 2023
@alattalatta
Copy link
Member

Done for ko. 😄

@leon-win
Copy link
Member

Done for ru 👌

@queengooborg queengooborg removed the l10n-ru Issues related to Russian content. label Dec 11, 2023
@github-project-automation github-project-automation bot moved this to ✅ Done in l10n-ko Aug 28, 2024
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
9 participants