Skip to content

docs: add new redirects to vercel.json#6266

Merged
matthewelwell merged 4 commits intomainfrom
docs-add-new-redirects
Nov 13, 2025
Merged

docs: add new redirects to vercel.json#6266
matthewelwell merged 4 commits intomainfrom
docs-add-new-redirects

Conversation

@robertnorrie
Copy link
Contributor

@robertnorrie robertnorrie commented Nov 11, 2025

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

Added docs redirects to vercel.json

How did you test this code?

N/A
Please describe.

@robertnorrie robertnorrie requested a review from a team as a code owner November 11, 2025 09:31
@robertnorrie robertnorrie requested review from matthewelwell and removed request for a team November 11, 2025 09:31
@vercel
Copy link

vercel bot commented Nov 11, 2025

Deployment failed with the following error:

Redirect at index 5 has invalid `source` pattern "/deployment/hosting/kubernetes?ref=awsmfoss.com".

Learn More: https://vercel.link/invalid-route-source-pattern

@vercel
Copy link

vercel bot commented Nov 11, 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 13, 2025 10:24am
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
flagsmith-frontend-preview Ignored Ignored Preview Nov 13, 2025 10:24am
flagsmith-frontend-staging Ignored Ignored Preview Nov 13, 2025 10:24am

@robertnorrie robertnorrie changed the title Add new redirects to vercel.json docs: add new redirects to vercel.json Nov 11, 2025
@github-actions github-actions bot added the docs Documentation updates label Nov 11, 2025
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Removed duplicate entries for staged rollouts and fixed error in kubernetes URL.
@axolo-co
Copy link

axolo-co bot commented Nov 11, 2025

robertnorrie commented:
I've fixed the issues Cursor highlighted. Please review when you have time @matthewelwell

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Duplicate Redirects: Streamline Configuration

Duplicate redirect for /clients/server-side exists at both line 488 and line 676 with identical destinations. The duplicate entry is redundant and could cause confusion during maintenance.

docs/vercel.json#L487-L489

flagsmith/docs/vercel.json

Lines 487 to 489 in d79ceba

{
"source": "/clients/server-side",
"destination": "/integrating-with-flagsmith/sdks/server-side"

Fix in Cursor Fix in Web


Bug: Redirect Duplication Causes Inconsistent Navigation.

Duplicate redirect for /basic-features exists at both line 508 and line 684, but with different destinations: line 509 has flagsmith-concepts/data-model (missing leading slash) while line 685 has /flagsmith-concepts/data-model. This creates inconsistent redirect behavior.

docs/vercel.json#L507-L509

flagsmith/docs/vercel.json

Lines 507 to 509 in d79ceba

{
"source": "/basic-features",
"destination": "flagsmith-concepts/data-model"

Fix in Cursor Fix in Web


Copy link
Contributor

@matthewelwell matthewelwell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a bunch of specific changes that are needed, but the formatting is also off - make sure you run the json linter.

Fixed errors and added new LD importer redirect.
@axolo-co
Copy link

axolo-co bot commented Nov 13, 2025

robertnorrie commented:
@matthewelwell I've added the changes and the missing LD redirect, and also checked the formatting is all ok.

@matthewelwell matthewelwell merged commit 0b87e6b into main Nov 13, 2025
12 checks passed
@matthewelwell matthewelwell deleted the docs-add-new-redirects branch November 13, 2025 10:27
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.

2 participants