Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Path priority issue #123

Closed
revathskumar opened this issue Jan 11, 2017 · 3 comments
Closed

Path priority issue #123

revathskumar opened this issue Jan 11, 2017 · 3 comments

Comments

@revathskumar
Copy link

In below routes, the url /about will navigate to <Mail path="/:id"/> instead of <About path="/about" />

Slack Discussion

<Router> 
  <Inbox path="/" default />              
  <About path="/about" />               
  <Mail path="/:id"/>            
</Router>

I am trying out fixes as per this comment But as of now no luck. Will update here is If I made any progress.

@revathskumar
Copy link
Author

To reproduce, here is the sample repo : https://github.com/revathskumar/react-inbox/tree/master/preact

@gabrielrtakeda
Copy link

gabrielrtakeda commented May 6, 2017

Any progress about this issue? I am facing the same problem..

<Router onChange={onChange}>
  <Home path="/" />
  <MemoryCreate path="/memory" />
  <Memory path="/:slug" />
  <Error404 default />
</Router>

gabrielrtakeda pushed a commit to gabrielrtakeda/preact-router that referenced this issue May 16, 2017
@developit
Copy link
Member

Just the referenced PR. Looking into it this week!

ashsearle added a commit to ashsearle/preact-router that referenced this issue Oct 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants