Skip to content

Update throw-on-field-error-directive.md #5039

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

christiansany
Copy link
Contributor

@christiansany christiansany commented Jul 22, 2025

Currently on https://relay.dev/docs/guides/throw-on-field-error-directive/, this slash at the end of the file name will result in a double slash when trying to visit the site it links to (-> link is broken).

This change will remove the double slash and make the link work.

Currently on https://relay.dev/docs/guides/throw-on-field-error-directive/, this slash at the end of the file name will result in a double slash when visiting the site.

This change will remove the double slash and make the link work.
@captbaritone
Copy link
Contributor

Interesting! I've seen other issues with broken links related to trailing slashes. Do you want to grep for all such /# links and replace them? At the very least we should update the non-versioned version of this page here:

slug: /guides/throw-on-field-error-directive/

Could you also confirm that the link still works by doing a full build of the docs and clicking the link (if you haven't already).

Thanks again for looking into this!

@captbaritone
Copy link
Contributor

Ping @christiansany

This was referenced Aug 1, 2025
@captbaritone
Copy link
Contributor

captbaritone commented Aug 2, 2025

I've opened #5049 in an attempt to fix this issue and others in historical versions of the docs.

facebook-github-bot pushed a commit that referenced this pull request Aug 7, 2025
Summary:
Expanding upon #5039 to catch all such instances.

Search for regex `\.mdx?/`.

Pull Request resolved: #5049

Test Plan: Ran the docs in dev mode and confirmed that I could click the link and get to the why-null page at the right anchor

Reviewed By: evanyeung

Differential Revision: D79477408

Pulled By: captbaritone

fbshipit-source-id: 1d2ce71714ba7d7309029081288cbb8e342f510d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants