Skip to content

feat: Use basePath for logo to fix loading issue#9

Merged
besessener merged 1 commit intomainfrom
fix-logo-path
Aug 14, 2025
Merged

feat: Use basePath for logo to fix loading issue#9
besessener merged 1 commit intomainfrom
fix-logo-path

Conversation

@besessener
Copy link
Copy Markdown
Contributor

Introduced a withBasePath utility function to correctly prepend the Next.js basePath to asset URLs.

This resolves an issue where the logo image in the header would not load when the application is deployed under a sub-path. The Header component was refactored to use this new utility, ensuring that the image path is always correct regardless of the deployment environment.

The new withBasePath function is located in web/src/app/utils/path.ts for reusability across the application.

Introduced a `withBasePath` utility function to correctly prepend the Next.js `basePath` to asset URLs.

This resolves an issue where the logo image in the header would not load when the application is deployed under a sub-path. The `Header` component was refactored to use this new utility, ensuring that the image path is always correct regardless of the deployment environment.

The new `withBasePath` function is located in `web/src/app/utils/path.ts` for reusability across the application.
@besessener besessener merged commit 72a7a0f into main Aug 14, 2025
1 check passed
@besessener besessener deleted the fix-logo-path branch August 14, 2025 07:28
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