Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Root router have a path different to '' or '/' #10852

Merged
merged 8 commits into from
Oct 2, 2023
Merged

Conversation

yracnet
Copy link
Contributor

@yracnet yracnet commented Sep 7, 2023

I've detected that getShortCircuitMatches is able to find a default or root router. However, there may be cases where this root router has a base path.

In my plugin, vite-plugin-remix, which supports base path deployment, I've encountered an issue related to these rules. I've searched for documentation or a definition regarding the "shim-error-route" but couldn't find any.

I'm submitting this PR to address this issue.

@changeset-bot
Copy link

changeset-bot bot commented Sep 7, 2023

🦋 Changeset detected

Latest commit: 9033e73

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@remix-run/router Patch
react-router Patch
react-router-dom Patch
react-router-dom-v5-compat Patch
react-router-native Patch

Not sure what this means? Click here to learn what changesets are.

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

Co-authored-by: Matt Brophy <matt@brophy.org>
@remix-cla-bot
Copy link
Contributor

remix-cla-bot bot commented Sep 29, 2023

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

@brophdawg11 brophdawg11 changed the base branch from main to dev October 2, 2023 15:34
@brophdawg11 brophdawg11 merged commit 8af53e7 into remix-run:dev Oct 2, 2023
@brophdawg11
Copy link
Contributor

Thank you!

@github-actions
Copy link
Contributor

🤖 Hello there,

We just published version 6.17.0-pre.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

@github-actions
Copy link
Contributor

🤖 Hello there,

We just published version 6.17.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants