yari icon indicating copy to clipboard operation
yari copied to clipboard

feature(tool/move): update files pointing to the moved document(s)

Open OnkarRuikar opened this issue 1 year ago • 3 comments

  • Fixes https://github.com/mdn/yari/issues/8292

  • Changes are almost same as https://github.com/mdn/yari/issues/8372

The PR:

  • updates move command to update referring documents.
  • adds status(new) and l10n(translated content) frontmatter attributes to optionalFrontMatterKeys array. Without this change Document.saveFile() removes these from metadata.
  • changed referringFiles type from array to set in delete command. When a dirctory is deleted multiple sub pages get deleted so the array was getting duplicate entries.

OnkarRuikar avatar Mar 25 '23 00:03 OnkarRuikar

Can we have a review on this? @Claas or @LeoMcA ? #8372 proved valuable in keeping the number of flaws at bay.

A few refactoring (like CSS) are ahead of us (as part of Mozilla's initiative to improve discoverability and navigability) and this feature will be needed to lower significantly the amount of menial work needed.

teoli2003 avatar Apr 12 '23 05:04 teoli2003

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

github-actions[bot] avatar Jul 14 '23 15:07 github-actions[bot]

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

github-actions[bot] avatar Mar 05 '24 15:03 github-actions[bot]

Closing because this has been implemented and thoroughly tested on content side.

OnkarRuikar avatar May 08 '24 06:05 OnkarRuikar