Comments (11)
The issue has resolved itself on my repository. Can't reproduce the error anymore, it might be related to something else.
from react-router-breadcrumbs-hoc.
@icd2k3 I see why it didn't work now - it wasn't clear to me that routesConfig
has to be passed to withBreadcrumbs()
🙂thought it should happen automatically or something. Anyway, this should do it, works fine.
from react-router-breadcrumbs-hoc.
Ohh, I could do a better job of calling that out in the readme... added a small change here.
Thanks @pimmey!
from react-router-breadcrumbs-hoc.
Hey @Pixelatex,
That should work. Could you post your entire route/breadcrumb config array when you have a moment?
Thanks
from react-router-breadcrumbs-hoc.
Got it work by moving the breadcrumb under the path like this:
{
path: '/careers/:id',
breadcrumb: 'test',
exact: true,
component: CareerDetail
},
No idea why this is an issue though, really weird bug
from react-router-breadcrumbs-hoc.
Strange! Did the order of the route config objects change by chance? The HOC will return the first match, so order can affect how the breadcrumbs are generated: https://github.com/icd2k3/react-router-breadcrumbs-hoc#order-matters
from react-router-breadcrumbs-hoc.
the order of the route objects wasn't an issue, I tried this on a few other routes and it also had the same issue somehow
from react-router-breadcrumbs-hoc.
Thanks for the follow-up @Pixelatex!
I adjusted some unit tests to see if I could repro the issue, but so far no dice! When you have a moment, would you be able to post your entire route array? It may help in tracking down the problem.
from react-router-breadcrumbs-hoc.
Same here, have you guys figured this one out by any chance? :-)
Tried moving the breadcrumb and what not, but nothing changes.
const ROUTES = [
{
path: '/yearbook',
component: Yearbook,
exact: true,
breadcrumb: 'Overview orders'
},
{
path: '/yearbook/:id',
component: Order,
breadcrumb: 'Overview orders 2'
}
]
from react-router-breadcrumbs-hoc.
Hey @pimmey, thanks for reporting and apologies for the issue. I'm re-opening this thread and I'll try to come up with a repro case tonight!
from react-router-breadcrumbs-hoc.
@pimmey I set up a quick code sandbox with your route config and it seems to be working alright... maybe something else is causing the issue? 🤔 are you using any of the option params like disabling defaults, etc?
https://codesandbox.io/s/pknr4kyll0
from react-router-breadcrumbs-hoc.
Related Issues (20)
- Translated breadcrumbs (delaying routes init) HOT 1
- dassad
- Unnecessary peer dependency on react-router-dom HOT 4
- "//" produces double matches HOT 1
- Unable to pass additional props. HOT 3
- define breadcrumb‘s description HOT 2
- Disabling default generated breadcrumbs by passing breadcrumb: null raises an error HOT 2
- Property 'match' does not exist on type 'ReactNode' HOT 1
- What to use instead of InjectedProps HOT 3
- Replacing react-router with react-router-dom HOT 2
- Fixed
- Suppress autmatically added Home Breadcrumb HOT 2
- Issue with excludePaths not being consistent with breadcrumbs: null HOT 6
- Question about type definition HOT 3
- typescript definition mistake HOT 1
- How do I add in between crumbs ? HOT 2
- Question: Is there a way to not export the current page in breadcrumbs HOT 2
- Question: breadcrumbs propTypes HOT 1
- How to remove "-" from titles HOT 3
- react-router v6 HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from react-router-breadcrumbs-hoc.