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 Markdown links in content when a page is deleted using command yarn content delete #8372

Closed
OnkarRuikar opened this issue Mar 8, 2023 · 1 comment · Fixed by #8412
Labels
enhancement Improves an existing feature. needs discussion p3 We don't have visibility when this will be addressed. wx writer experience

Comments

@OnkarRuikar
Copy link
Contributor

Related to:

Problem

After using the deletion script links pointing the deleted document remain in content. And info around the links become invalid and go unaddressed. Over time content accumulates a lot of broken links and each link requires manual edit. It would be great if the deletion script automatically deletes the links or at least report them so that contributor will clean them.

Solution

a. Automatically delete all en-US/docs/${deletedDocPath} from content. And log 10 documents were referring to the deleted doc. Check and update info around it before commit.
b. Simply log doc paths and location referring the deleted doc.

With approach b. the contributor will have to manually search the documents and location to update which could be lengthy. With approach a. they simply will have to look at the diff and do additional edits around the deleted link which is more time saving.

@github-actions github-actions bot added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Mar 8, 2023
@teoli2003
Copy link
Contributor

I would prefer that it reports them. Deleting them, or making them inactive, will not fix the text around them.

@caugner caugner added enhancement Improves an existing feature. p3 We don't have visibility when this will be addressed. accepting PR We invite you to open a PR to resolve this issue. wx writer experience needs discussion and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. accepting PR We invite you to open a PR to resolve this issue. labels Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves an existing feature. needs discussion p3 We don't have visibility when this will be addressed. wx writer experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants