Optional Url Params

Hi,

Love the project; thanks,

I have a problem with a path with an optional parameter.

When navigating to /sensor/register the breadcrumb shows:

Register Sensor

but navigating to /sensor/register/123 results in:

Register Sensor / Register Sensor

I have the following path setup in my routes.config - the offending route is at the bottom…

const routesConfig = [
  {
    path: '/hubs',
    exact: true,
    component: MyHubsContainer,
    breadcrumb: 'My Hubs'
  },
  {
    path: '/hubs/register',
    exact: true,
    component: RegisterHubContainer,
    breadcrumb: 'Register Hub'
  },
  {
    path: '/hubs/:hubId',
    exact: true,
    component: HubDashboardContainer,
    breadcrumb: HubBreadcrumbItem
  },
  {
    path: '/hubs/:hubId/sensor/:sensorId/:sensorName',
    exact: true,
    component: SensorDetailContainer,
    breadcrumb: SensorBreadcrumbItem
  },
  {
    path: '/sensor/instructions',
    exact: true,
    component: InstructionsContainer
  },
  {
    path: '/sensor/register/:hubId?',
    exact: true,
    component: RegisterSensorContainer,
    breadcrumb: 'Register Sensor'
  }
]

const MainRoutes = (props) =>
  <Switch>
    {routesConfig.map((route, i) =>
      <Route
        exact={route.exact}
        key={route.path}
        path={route.path}
        component={route.component}
      />)}
  </Switch>

const Breadcrumbs = ({ breadcrumbs }) =>
  <div>
    {breadcrumbs.map((breadcrumb, index) => (
      <Link key={breadcrumb.key} to={breadcrumb.props.match.url || ''}>
        {breadcrumb} {console.log('breadcrumb', breadcrumbs.length, breadcrumb)}
      </Link>
    ))}
  </div>

Any ideas?

[icd2k3]

Hey @luke-unifymonitor this key warning is fixed in 2.1.4. Here is the updated code sandbox: https://codesandbox.io/s/13j7vr75yl

[icd2k3]

🤔 I should fix that warning. I’ll look into this case in the near future.

Thanks for the reply and the code sandbox @luke-unifymonitor !