Skip to content

docs: handle trailing slash behaviour for all URLs#6254

Merged
matthewelwell merged 1 commit intomainfrom
docs/better-handling-of-trailing-slashes
Nov 7, 2025
Merged

docs: handle trailing slash behaviour for all URLs#6254
matthewelwell merged 1 commit intomainfrom
docs/better-handling-of-trailing-slashes

Conversation

@matthewelwell
Copy link
Contributor

@matthewelwell matthewelwell commented Nov 7, 2025

Changes

The PR here #6252 removed a load of 'duplicates', but actually they were working around the fact that the default configuration in vercel doesn't handle trailing slash re-routing. This PR means that any URL with a trailing slash will be re-routed to the naked URI (i.e. strip the trailing slash).

How did you test this code?

Tested via preview:

  1. Tested a few redirect URLs (e.g. /clients/ and /clients, /clients/ios/ and /clients/ios) to confirm that the redirects work as expected now
  2. Smoke tested the docs to verify links around the site all work as expected

@vercel
Copy link

vercel bot commented Nov 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Nov 7, 2025 3:24pm
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
flagsmith-frontend-preview Ignored Ignored Nov 7, 2025 3:24pm
flagsmith-frontend-staging Ignored Ignored Nov 7, 2025 3:24pm

@github-actions github-actions bot added the docs Documentation updates label Nov 7, 2025
@matthewelwell matthewelwell marked this pull request as ready for review November 7, 2025 15:28
@matthewelwell matthewelwell requested a review from a team as a code owner November 7, 2025 15:28
@matthewelwell matthewelwell requested review from robertnorrie and removed request for a team November 7, 2025 15:28
@matthewelwell matthewelwell merged commit fe95955 into main Nov 7, 2025
14 checks passed
@matthewelwell matthewelwell deleted the docs/better-handling-of-trailing-slashes branch November 7, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants