Skip to content

Conversation

@Linho1219
Copy link
Member

移除 base,直接从根解析

移除 base,直接从根解析
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the basePath configuration from the Next.js documentation site, changing it to resolve from the root path instead of the /applemusic-like-lyrics subdirectory.

Changes:

  • Removed basePath: "/applemusic-like-lyrics" from Next.js configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

output: 'export',
reactStrictMode: true,
basePath: "/applemusic-like-lyrics",
pageExtensions: ["js", "jsx", "mdx", "ts", "tsx"],
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

Removing the basePath requires updating hardcoded paths in the codebase. The following files still contain the old /applemusic-like-lyrics prefix: src/app/[lang]/page.tsx (line 13, Image src), src/app/layout.tsx (line 8, icons path), and README.md (line 5, documentation link). These paths should be updated to remove the /applemusic-like-lyrics prefix and use root-relative paths instead.

Copilot uses AI. Check for mistakes.
output: 'export',
reactStrictMode: true,
basePath: "/applemusic-like-lyrics",
pageExtensions: ["js", "jsx", "mdx", "ts", "tsx"],
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The README.md at line 5 still references the old URL path with /applemusic-like-lyrics. If the site will be deployed to a different location or root path, this documentation link needs to be updated to reflect the new deployment URL.

Copilot uses AI. Check for mistakes.
@Linho1219 Linho1219 closed this Jan 16, 2026
@Linho1219 Linho1219 deleted the Linho1219-patch-1 branch January 16, 2026 15:44
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.

2 participants