Skip to content

Fixed 404s #411

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.

Sign up for GitHub

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

Merged
sriramveeraghanta merged 1 commit into from
Mar 12, 2026
Merged

Fixed 404s #411

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 30 additions & 2 deletions vercel.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,18 @@
"source": "/quick-start",
"destination": "/introduction/quickstart"
},
{
"source": "/quickstart",
"destination": "/introduction/quickstart"
},
{
"source": "/sign-up-sign-in",
"destination": "/introduction/signup-signin"
},
{
"source": "/introduction/signup-signin",
"destination": "/introduction/home"
},
Comment on lines 13 to +20
Copy link

@coderabbitai coderabbitai bot Mar 12, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Redirect chain detected.

Adding this redirect creates a 3-hop chain:
/sign-up-sign-in → /introduction/signup-signin → /introduction/home → /

Multi-hop redirects degrade performance and SEO. Consider updating the existing redirect on lines 14-15 to point directly to the final destination.

🔧 Proposed fix to flatten the redirect chain 
     {
       "source": "/sign-up-sign-in",
-      "destination": "/introduction/signup-signin"
+      "destination": "/"
     },
     {
       "source": "/introduction/signup-signin",
-      "destination": "/introduction/home"
+      "destination": "/"
     },
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@vercel.json` around lines 13 - 20, The redirect from "/sign-up-sign-in"
currently points to "/introduction/signup-signin" which itself redirects to
"/introduction/home", creating a chain; update the redirect entry where source
is "/sign-up-sign-in" so its destination is the final target (e.g., "/") instead
of "/introduction/signup-signin" to flatten the chain (or remove the
intermediate "/introduction/signup-signin" redirect if you want a single
mapping).

{
"source": "/docker-compose",
"destination": "/self-hosting/docker-compose"
Expand All @@ -30,10 +38,18 @@
"source": "/workspaces",
"destination": "/core-concepts/workspaces"
},
{
"source": "/core-concepts/workspaces",
"destination": "/core-concepts/workspaces/overview"
},
Comment on lines 38 to +44
Copy link

@coderabbitai coderabbitai bot Mar 12, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Redirect chain detected.

Adding this redirect creates a 2-hop chain:
/workspaces → /core-concepts/workspaces → /core-concepts/workspaces/overview

Consider updating the existing redirect to point directly to the final destination.

🔧 Proposed fix to flatten the redirect chain 
     {
       "source": "/workspaces",
-      "destination": "/core-concepts/workspaces"
+      "destination": "/core-concepts/workspaces/overview"
     },
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@vercel.json` around lines 38 - 44, The redirect entries create a 2-hop chain:
an object with "source": "/workspaces" currently points to "destination":
"/core-concepts/workspaces" and there's another object redirecting
"/core-concepts/workspaces" to "/core-concepts/workspaces/overview"; update the
first redirect so its "destination" is the final path
"/core-concepts/workspaces/overview" (or remove the intermediate redirect),
ensuring the "source" "/workspaces" maps directly to the final destination and
eliminating the redundant "/core-concepts/workspaces" redirect.

{
"source": "/projects",
"destination": "/core-concepts/projects"
},
{
"source": "/core-concepts/projects",
"destination": "/core-concepts/projects/overview"
},
Comment on lines 46 to +52
Copy link

@coderabbitai coderabbitai bot Mar 12, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Redirect chain detected.

Same pattern as workspaces: /projects → /core-concepts/projects → /core-concepts/projects/overview

Consider updating the existing redirect to point directly to the final destination.

🔧 Proposed fix to flatten the redirect chain 
     {
       "source": "/projects",
-      "destination": "/core-concepts/projects"
+      "destination": "/core-concepts/projects/overview"
     },
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"source": "/projects",
"destination": "/core-concepts/projects"
},
{
"source": "/core-concepts/projects",
"destination": "/core-concepts/projects/overview"
},
"source": "/projects",
"destination": "/core-concepts/projects/overview"
},
{
"source": "/core-concepts/projects",
"destination": "/core-concepts/projects/overview"
},
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@vercel.json` around lines 46 - 52, Update the redirect entry that maps
"source": "/projects" (currently pointing to "destination":
"/core-concepts/projects") to point directly at the final target "destination":
"/core-concepts/projects/overview" to remove the redirect chain; locate the two
redirect objects referencing "/projects" and "/core-concepts/projects" and
change the first object's destination to "/core-concepts/projects/overview".

{
"source": "/issues",
"destination": "/core-concepts/issues"
Expand Down Expand Up @@ -66,6 +82,10 @@
"source": "/power-k",
"destination": "/core-concepts/power-k"
},
{
"source": "/integrations",
"destination": "/integrations/about"
},
{
"source": "/plane-integrations/about",
"destination": "/integrations/about"
Expand All @@ -82,6 +102,10 @@
"source": "/importers/github-imp",
"destination": "/importers/github"
},
{
"source": "/importers/github",
"destination": "/importers/overview"
},
Comment on lines 102 to +108
Copy link

@coderabbitai coderabbitai bot Mar 12, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check what importer documentation files exist
echo "=== Importer documentation files ==="
fd -e md . docs/importers 2>/dev/null || find docs/importers -name "*.md" 2>/dev/null

echo -e "\n=== Check for github-related importer files ==="
fd -i github -e md . docs 2>/dev/null || find docs -iname "*github*" -name "*.md" 2>/dev/null

Repository: makeplane/docs

Length of output: 491

Redirect chain loses GitHub-specific documentation.

The redirect chain /importers/github-imp → /importers/github → /importers/overview is problematic. Since github-imp.md exists with the actual GitHub Importer documentation, users following this chain will end up on the generic /importers/overview page instead of the GitHub-specific documentation.

Either remove the /importers/github redirect if it's not needed, or update it to point directly to /importers/github-imp to preserve access to the GitHub Importer documentation.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@vercel.json` around lines 102 - 108, The redirect chain defined for the route
with "source": "/importers/github-imp" and the subsequent rule for "source":
"/importers/github" sends users to "/importers/overview", which loses the
GitHub-specific page; update the redirect rules so that the "/importers/github"
entry either is removed (if redundant) or changed to point to
"/importers/github-imp" instead of "/importers/overview" to ensure requests for
"/importers/github" resolve to the GitHub Importer documentation.

{
"source": "/importers/jira-imp",
"destination": "/importers/jira"
Expand All @@ -90,6 +114,10 @@
"source": "/webhooks/introduction",
"destination": "/webhooks/intro-webhooks"
},
{
"source": "/api",
"destination": "/api-reference/introduction"
},
{
"source": "/apis/introduction",
"destination": "/api-reference/introduction"
Expand Down Expand Up @@ -204,7 +232,7 @@
},
{
"source": "/webhooks/:path*",
"destination": "https://developers.plane.so/webhooks/:path*"
"destination": "https://developers.plane.so/dev-tools/intro-webhooks/:path*"
},
Comment on lines 233 to 236
Copy link

@coderabbitai coderabbitai bot Mar 12, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Test if the external webhook URLs are accessible
echo "Testing external webhook redirect destinations..."

# Test base intro-webhooks page
curl -sI "https://developers.plane.so/dev-tools/intro-webhooks" | head -5

# Test a potential subpath
curl -sI "https://developers.plane.so/dev-tools/intro-webhooks/test" | head -5

Repository: makeplane/docs

Length of output: 364

Remove the :path* wildcard from the redirect destination.

The external URL https://developers.plane.so/dev-tools/intro-webhooks/ does not support nested subpaths. Testing shows the base path returns HTTP 200, but any subpath (e.g., /test) returns HTTP 404. This redirect rule will cause 404s for any request to /webhooks/:path*.

Change the destination to https://developers.plane.so/dev-tools/intro-webhooks without the wildcard appending:

{
  "source": "/webhooks/:path*",
  "destination": "https://developers.plane.so/dev-tools/intro-webhooks"
}
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@vercel.json` around lines 233 - 236, The redirect rule using source
"/webhooks/:path*" currently appends the ":path*" wildcard to the external
destination, causing 404s for nested subpaths; update the rule so the
destination for that redirect is the fixed external URL
"https://developers.plane.so/dev-tools/intro-webhooks" (remove the ":path*" from
the destination) while keeping the source unchanged to ensure all /webhooks
requests map to the base docs page.

{
"source": "/plane-one/governance/workspaces-and-teams",
Expand All @@ -231,7 +259,7 @@
"destination": "/importers/jira"
},
{
"source": "/performance/hyper-mode#use-hyper-mode",
"source": "/performance/hyper-mode",
"destination": "/"
}
]
Expand Down
Loading