React Router work on reload, but not when clicking on a link

Question

I have setup the React with react-router version 4. The routing works when I enter the URL directly on the browser, however when I click on the link, the URL changes on the browser (e.g http://localhost:8080/categories), but the content don't get updated (But if I do a refresh, it gets updated).

Below is my setup:

The Routes.js setup as follows:

import { Switch, Route } from 'react-router-dom';
import React from 'react';

// Components
import Categories from './containers/videos/Categories';
import Videos from './containers/videos/Videos';
import Home from './components/Home';

const routes = () => (
  <Switch>
    <Route exact path="/" component={Home}/>
    <Route path="/videos" component={Videos}/>
    <Route path="/categories" component={Categories}/>
  </Switch>
);

export default routes;

The link I use in Nav.js are as follows:

<Link to="/videos">Videos</Link>
<Link to="/categories">Categories</Link>

The App.js is as follows:

import React from 'react';
import './app.scss';
import Routes from './../routes';
import Nav from './Nav';

class AppComponent extends React.Component {

  render() {
    return (
      <div className="index">
        <Nav />
        <div className="container">
          <Routes />
        </div>
      </div>
    );
  }
}

AppComponent.defaultProps = {
};

export default AppComponent;

Did you get your problem solved? I have exactly the same issue and I'm interested in your final solution. — dns_nx, Apr 27 '18 at 08:17
@dns_nx The problem I had was putting in the in the wrong place. The correct would be as in this pastebin: https://pastebin.com/M9hU4Bg4 — Suthan Bala, Apr 27 '18 at 12:52
@SuthanBala - your latest commet saved me - after 6Hrs (?!) of spinning in circles. I was not using BrowseRouter at all ... just Router. Jeez ... — Liviu Sosu, Dec 10 '20 at 13:12

score 17 · Answer 1 · answered Sep 14 '17 at 06:42

17

I would go through your components and make sure you have only one <Router> ... </Router>. Also -- make sure you have a <Router>...</Router> There may be cases when you'd use more than one, but if you accidentally have nested routers (because you were hacking quickly and forgot to remove one when you were moving it around to all kinds of places ;-) - it could cause an issue.

I would try

import {
  BrowserRouter as Router,
}  from 'react-router-dom'

// Other Imports

...

return (
  <Router>
    <div className="index">
      <Nav /> <!-- In this component you have <Links> -->
      <div className="container">
        <Routes />
      </div>
    </div>
  </Router>
);

In your top most component (App.js).

answered Sep 14 '17 at 06:42

Peege151

1,518
1
20
40

1

Double `` was the issue in my case. – Cezar D. Feb 27 '19 at 22:05
2

switching to Browserrouter instead of just Router worked for me! thanks – Max Jul 20 '20 at 15:08
This solution works for me. I used double Router in my app. – Joxon Nov 03 '20 at 02:29

Shubham Khatri · Accepted Answer · 2018-01-06T04:41:06.387

Wrapping your component with withRouter should do the job for you. withRouter is needed for a Component that uses Link or any other Router props and doesn't receive the Router props either directly from Route or from the Parent Component

Router Props are available to the component when its called like

<Route component={App}/>

or

<Route render={(props) => <App {...props}/>}/>

or if you are placing the Links as direct children of Router tag like

<Router>
     <Link path="/">Home</Link>
</Router>

In case when you wish to write the child content within Router as a component, like

<Router>
     <App/>
</Router>

The Router props won't be available to App and hence, you could pass call it using a Route like

<Router>
     <Route component={App}/>
</Router>

However withRouter comes in Handy when you want to provide the Router props to a highly nested component. Check this solution

import {withRouter} from 'react-router'

class AppComponent extends React.Component {

  render() {
    return (
      <div className="index">
        <Nav />
        <div className="container">
          <Routes />
        </div>
      </div>
    );
  }
}

AppComponent.defaultProps = {
};

export default withRouter(AppComponent);

Nvm, I had the `browserhistory` in the wrong location. Adjusting that fixed it. — Suthan Bala, Jun 04 '17 at 20:12
where did you adjust it, i'm have same problem, can you post a full solution? — Zhang Bruce, Aug 21 '17 at 04:25
If using connect from react-redux, then wrap the withRouter around that connect. ex: `export default withRouter(connect(mapStateToProps)(App))`. Solution for that here https://stackoverflow.com/a/45082976/3714697 — larrydalmeida, Sep 07 '17 at 03:37

score 4 · Answer 3 · answered Oct 07 '19 at 22:41

4

There should be only one ROUTER in the whole app, which I think if your head file is App.js, then the ROUTER should wrap the whole App.js component.

answered Oct 07 '19 at 22:41

Boanerges

45
2

score 3 · Answer 4 · answered Nov 26 '18 at 06:09

I was having the exact same issue, but the cause of mine was much simpler.

In my case I had a space at the end of the URL string:

<Link to={ "/myentity/" + id + "/edit " } >Edit</Link>

Wouldn't work when I clicked on the link, but the URL would update in the address bar. Tapping on the browser address bar and hitting the enter key would then work correctly.

Removing the space fixed it:

<Link to={ "/myentity/" + id + "/edit" } >Edit</Link>

Pretty obvious I guess, but easy to overlook. A few hairs were pulled before I noticed (and clearly I ended up here), hope it saves someone else a few hairs.

Thank you! Very easily overlooked – Jari Pekkala Mar 16 '19 at 11:54 — Jari Pekkala, Mar 16 '19 at 11:54

score 0 · Answer 5 · answered Jun 04 '19 at 00:04

0

I was facing this because I have not installed the react-router in the application (package.json) dependencies...

answered Jun 04 '19 at 00:04

Gaspar

1,177
10
19

score 0 · Answer 6 · answered Mar 18 '21 at 04:15

0

You should not wrap the nav-bar where your are stored in or

I also had the same issue before. The mistake was the I wrapped my navbar component inside a you should use in the main component only (App.js)

answered Mar 18 '21 at 04:15

ADITHYA AJAY

39
6

React Router work on reload, but not when clicking on a link

6 Answers6

Linked