Skip to content

docs: Correct NCMEC docs link#169

Merged
cassidyjames merged 1 commit intomainfrom
cassidyjames/docs-ncmec-link
Mar 31, 2026
Merged

docs: Correct NCMEC docs link#169
cassidyjames merged 1 commit intomainfrom
cassidyjames/docs-ncmec-link

Conversation

@cassidyjames
Copy link
Copy Markdown
Member

This was using an absolute path which didn't work for the mdbook version. Switch to relative, and make the link name consistent with the target page title.

This was using an absolute path which didn't work for the mdbook version. Switch to relative, and make the link name consistent with the target page title.
@cassidyjames cassidyjames requested a review from a team as a code owner March 31, 2026 15:13
@cassidyjames
Copy link
Copy Markdown
Member Author

Actually, looking at this closer, I think this whole section is redundant with the NCMEC page. @julietshen would it make sense to remove this section here since we have a dedicated page?

@julietshen
Copy link
Copy Markdown
Member

let's keep it - it's redundant but I think ok to have this in more places.

@cassidyjames
Copy link
Copy Markdown
Member Author

cassidyjames commented Mar 31, 2026

@julietshen ack. I might clean up the first instance of it to be more of a summary rather than a (mostly worse-formatted) copy of the dedicated NCMEC file, but keeping it there for discoverability. I'll tackle that as a follow-up PR though.

@cassidyjames cassidyjames added the documentation Improvements or additions to documentation label Mar 31, 2026
@cassidyjames cassidyjames merged commit 94295e4 into main Mar 31, 2026
10 checks passed
@cassidyjames cassidyjames deleted the cassidyjames/docs-ncmec-link branch March 31, 2026 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants