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

23,366

Solution 1

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);

Solution 2

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).

Solution 3

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.

Solution 4

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.

Solution 5

If anyone is using the history package for navigation and if you are using the react-router v5, then you may have to downgrade the history package to 4.10.1 until the history package developers issue a new release with the fix. As of writing this, the latest release version of history package is v5.0.1, and if you see a new release than this, you can try with that version first to see if it works, before doing the downgrade to 4.10.1

Share:
23,366
Suthan Bala
Author by

Suthan Bala

Updated on November 19, 2021

Comments

  • Suthan Bala
    Suthan Bala over 2 years

    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;
    
  • Suthan Bala
    Suthan Bala almost 7 years
    still no luck :(
  • Suthan Bala
    Suthan Bala almost 7 years
    Nvm, I had the browserhistory in the wrong location. Adjusting that fixed it.
  • Zhang Bruce
    Zhang Bruce over 6 years
    where did you adjust it, i'm have same problem, can you post a full solution?
  • larrydalmeida
    larrydalmeida over 6 years
    If using connect from react-redux, then wrap the withRouter around that connect. ex: export default withRouter(connect(mapStateToProps)(App)). Solution for that here stackoverflow.com/a/45082976/3714697
  • Cezar D.
    Cezar D. about 5 years
    Double <Router/> was the issue in my case.
  • Jari Pekkala
    Jari Pekkala about 5 years
    Thank you! Very easily overlooked
  • Max
    Max almost 4 years
    switching to Browserrouter instead of just Router worked for me! thanks
  • Joxon
    Joxon over 3 years
    This solution works for me. I used double Router in my app.