Skip to content

fix(docs): adjust asset path#7507

Merged
mhartington merged 1 commit intomainfrom
fix/asset-paths
Feb 17, 2026
Merged

fix(docs): adjust asset path#7507
mhartington merged 1 commit intomainfrom
fix/asset-paths

Conversation

@mhartington
Copy link
Member

@mhartington mhartington commented Feb 17, 2026

Summary by CodeRabbit

  • Chores
    • Updated Next.js configuration settings for the documentation app, including asset serving and path handling.
    • Applied formatting adjustments to TypeScript configuration.

@vercel
Copy link

vercel bot commented Feb 17, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Feb 17, 2026 11:17am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 17, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates the docs app's Next.js configuration to explicitly set basePath to an empty string and conditionally set assetPrefix based on NODE_ENV. A commented-out redirects block is removed. Array formatting in tsconfig.json is standardized with consistent spacing.

Changes

Cohort / File(s) Summary
Next.js Configuration
apps/docs/next.config.mjs
Explicitly set basePath to empty string; updated assetPrefix to conditionally use '/docs' in production or empty string otherwise. Removed commented-out redirects() block.
TypeScript Configuration
apps/docs/tsconfig.json
Added consistent spacing inside lib and exclude arrays for formatting alignment.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

🍈 Lychee Link Check Report

3660 links: ✅ 2973 OK | 🚫 21 errors | 🔀 0 redirects | 👻 664 excluded

❌ Errors

./apps/docs/content/docs.v6/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs.v6/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs.v6/ai/index.mdx

./apps/docs/content/docs.v6/guides/solid-start.mdx

./apps/docs/content/docs.v6/orm/more/comparisons/prisma-and-drizzle.mdx

./apps/docs/content/docs.v6/orm/more/dev-environment/editor-setup.mdx

  • [503] https://fig.io/ | Rejected status code (this depends on your "accept" configuration): Service Unavailable

./apps/docs/content/docs.v6/orm/overview/introduction/should-you-use-prisma.mdx

./apps/docs/content/docs.v6/orm/overview/prisma-in-your-stack/graphql.mdx

./apps/docs/content/docs.v6/orm/prisma-client/setup-and-configuration/databases-connections/index.mdx

./apps/docs/content/docs.v6/orm/prisma-client/testing/integration-testing.mdx

./apps/docs/content/docs.v6/orm/prisma-client/testing/unit-testing.mdx

./apps/docs/content/docs.v6/orm/prisma-schema/overview/generators.mdx

./apps/docs/content/docs/(index)/prisma-orm/add-to-existing-project/mongodb.mdx

./apps/docs/content/docs/(index)/prisma-orm/quickstart/mongodb.mdx

./apps/docs/content/docs/ai/index.mdx

./apps/docs/content/docs/guides/frameworks/solid-start.mdx

./apps/docs/content/docs/orm/more/comparisons/prisma-and-drizzle.mdx

./apps/docs/content/docs/orm/more/dev-environment/editor-setup.mdx

  • [503] https://fig.io/ | Rejected status code (this depends on your "accept" configuration): Service Unavailable

./apps/docs/content/docs/orm/prisma-client/testing/integration-testing.mdx

./apps/docs/content/docs/orm/prisma-client/testing/unit-testing.mdx

./apps/docs/content/docs/orm/prisma-schema/overview/generators.mdx


Full Statistics Table
Status Count
✅ Successful 2973
🔀 Redirected 0
👻 Excluded 664
🚫 Errors 21
⛔ Unsupported 2
⏳ Timeouts 0
❓ Unknown 0

@mhartington mhartington merged commit 8893dd0 into main Feb 17, 2026
5 of 10 checks passed
@mhartington mhartington deleted the fix/asset-paths branch February 17, 2026 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant