Skip to content

update dependencies and fix issues to laod quick-example#3

Open
parthpatelsj wants to merge 1 commit intomainfrom
fix-quick-example
Open

update dependencies and fix issues to laod quick-example#3
parthpatelsj wants to merge 1 commit intomainfrom
fix-quick-example

Conversation

@parthpatelsj
Copy link

@parthpatelsj parthpatelsj commented Jan 29, 2026

  • React Router is refusing to render because the Router’s basename includes a trailing slash, but the current URL does not. With react-router-dom v7, basename must be an exact prefix of location.pathname. If basename is “/webspatial/avp/” and the URL is “/webspatial/avp”, they don’t match, so nothing renders.
  • Update dependencies
Screenshot 2026-01-29 at 5 29 17 PM

@changeset-bot
Copy link

changeset-bot bot commented Jan 29, 2026

⚠️ No Changeset found

Latest commit: 76ae6ae

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR


return (
<Router basename={__XR_ENV_BASE__}>
<Router basename={__XR_ENV_BASE__.replace(/\/$/, "")}>

Choose a reason for hiding this comment

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

When running on Vite dev server the path should be /webspatial/avp/ so doesn't that match the react-router configuration. When using /webspatial/avp assets from the public folder will not be loaded correctly.

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.

3 participants