Skip to content

Remove all redirects to other locales#36811

Merged
caugner merged 1 commit intomainfrom
MP-1719-remove-locale-redirects
Nov 18, 2024
Merged

Remove all redirects to other locales#36811
caugner merged 1 commit intomainfrom
MP-1719-remove-locale-redirects

Conversation

@argl
Copy link
Copy Markdown
Contributor

@argl argl commented Nov 15, 2024

Description

Removes all redirects into translated content that use targets with translated slugs, are outdated and/or invalid.
(MP-1719)

Motivation

rari/yari parity, rari does run consistency checks on redirects.

@github-actions github-actions bot added the size/xs [PR only] 0-5 LoC changed label Nov 15, 2024
@argl argl marked this pull request as ready for review November 15, 2024 14:28
@argl argl requested a review from a team as a code owner November 15, 2024 14:28
@argl argl requested review from hamishwillee and removed request for a team November 15, 2024 14:28
@hamishwillee hamishwillee requested a review from caugner November 18, 2024 01:22
@hamishwillee
Copy link
Copy Markdown
Collaborator

hamishwillee commented Nov 18, 2024

Thanks @argl .

Looking in the translation repo it looks like the translation redirects are generated from the English version (or similar) so this would make sense. However I'm not familiar with the process, so removing myself from review and pointing this at @caugner

@hamishwillee hamishwillee removed their request for review November 18, 2024 01:26
@caugner caugner changed the title removes redirects that use localized slugs into translated content Remove all redirects to other locales Nov 18, 2024
@caugner caugner merged commit f251163 into main Nov 18, 2024
@caugner caugner deleted the MP-1719-remove-locale-redirects branch November 18, 2024 10:51
wbamberg added a commit to wbamberg/content that referenced this pull request Nov 27, 2024
* origin/xss-guide: (284 commits)
  Add information on default entryPoint property values (mdn#36633)
  Bump husky from 9.1.6 to 9.1.7 (mdn#36863)
  fix(performance): Typo '50ms seconds' (mdn#36861)
  Add spec_url & add note for bcd for `<frequency>` and `<frequency-percentage>` (mdn#36848)
  addresses 36583: summary icon styles (mdn#36691)
  Remove "simple" part 3: change to "basic"  (mdn#36762)
  the default option of a select (mdn#36658)
  docs(css): Add support for `<string>` in `syntax` descriptor of @Property at-rule (mdn#36655)
  Fix parameter syntax for `Navigation.updateCurrentEntry()` (mdn#36852)
  Update CSP source expression reference (mdn#36792)
  chore(http): Refresh headers docs (d-k) (mdn#36075)
  chore(http): Refresh headers r-s (mdn#36590)
  Updated index.md (mdn#36845)
  fix : wrong method name (mdn#36843)
  Remove all redirects to other locales (mdn#36811)
  fix typos (mdn#36837)
  docs: update Accept-Charset status (mdn#36822)
  updateSelection: make more intuitive (mdn#36834)
  updateText: Remove false information (mdn#36832)
  DOMRect instance properties (mdn#36704)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants