Replace deprecated props.url in readme

[lucleray]

Following #4950

Fix #4716

I'm not sure to understand this part, though :

Note: in order to programmatically change the route without triggering navigation and component-fetching, use props.url.push and props.url.replace within a component

Is there a difference between :

export default () => <a onClick={() => Router.push('/about')}>About</a>

and

export default withRouter(
  () => <a onClick={() => props.router.push('/about')}>About</a>
)

?

[timneutkens]

Note: in order to programmatically change the route without triggering navigation and component-fetching, use props.url.push and props.url.replace within a component

This has been deprecated for ages (Since 2.0 was released or so).

In general, I would recommend using withRouter for everything as that follows React's component model vs import Router from 'next/router' being stateful/global.

[lucleray]

@timneutkens

Ok, I'll remove the note and I'll update the examples in the Readme accordingly 👍

[timneutkens]

Lets change this to use withRouter instead 👍

[lucleray]

I still feel like the Router API is a bit hidden in the part about "Intercepting popstate".

I think it would be good to move it to "Imperatively" just above, or even to move it to "withRouter", since it's the recommended way of using the router 😄