url changes before confirmLeave is satisfied
Opened this issue · 6 comments
confirmLeave is a nice feature!
Anyhow, when moving away from that route, url is changed before confirmLeave
is accepted. That should not happen. Apparently confirmLeave
is called after url has already been changed when it should block url change altogether until accepted.
I don't have this issue with confirmLeave: () => 'Are u sure?'
. The route action is blocked and the url stays the same. Are you returning a truthy value from confirmLeave
?
By returning truthy value, everything works almost like expected. With falsy value url is changed. The result is that url has changed but redux is not updated.
To emphasize, the url is changed before the popup appears, not when returning value from popup.
confirmLeave
needs to be a synchronous function - is that the case here?
Here is where the handler is created to check confirmation if you enter a route where you have configured confirmLeave
(in _afterRouteChange
):
redux-first-router/src/connectRoutes.js
Lines 420 to 429 in 0a3ff66
Here is where it blocks the navigation in _beforeRouteChange
if the handler has been set up for the current route (before the route change):
redux-first-router/src/connectRoutes.js
Lines 321 to 330 in 0a3ff66
If _beforeRouteChange
returns true, then the middleware should skip dispatching the action and changing the URL:
redux-first-router/src/connectRoutes.js
Lines 298 to 304 in 0a3ff66
Perhaps you could work out which part is going wrong?
I ran into this myself - it happens when you return false
from confirmLeave
. When reaching this line:
The redux action is skipped after callback(false)
happens (callback(true)
seems to be the expected behaviour to allow the transition to occur normally)
You can work around the issue by returning a falsy value other than false
(e.g. undefined
) from confirmLeave
, although I'd say its definitely a bug. Not sure what the purpose of that line is though.
I’ll ask James.
Will one of you place this into the replace the documentation. It’s likely to show up again as an issue haha.
I’m on my mobile for a few days. But will do it if I don’t see a PR ❤️❤️ Well done on the debugging efforts
@hedgepigdaniel would you mind opening a small or about this?