Skip to content

Conversation

rasheed8123
Copy link

…ployed link and locally

@rasheed8123 rasheed8123 reopened this Jun 28, 2025
@rasheed8123
Copy link
Author

Hi! 👋

This PR fixes static asset path issues for static deployments (like Netlify) by:

  • Removing the /ui prefix from all image and asset paths in the codebase.
  • Updating next.config.ts to use basePath and assetPrefix only for GitHub Pages (via DEPLOY_TARGET=github-pages), and defaulting to root for other hosts.
  • Replacing the Next.js component with a regular for the logo to ensure compatibility with static exports.

All changes have been tested with a static export and verified to work on Netlify.

Could you please review if this approach aligns with your deployment strategy?
If you have a preferred way to handle multi-environment static asset paths, I’m happy to adjust!

Thanks for your time and for maintaining this awesome project! 🚀

@stabldev
Copy link
Member

hi @rasheed8123, I managed to deploy site to vercel via github-actions, so we dont need to mess with paths anymore.
link: https://ui.headcn.site

could you rebase this PR?
and remove the /ui prefix for media files? they wont work as we changed the domain.
then we can merge this.

@stabldev stabldev added the invalid This doesn't seem right label Jul 28, 2025
@stabldev stabldev marked this pull request as draft July 28, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants