`onAfterChange` and redirects
See original GitHub issueBefore diving into a PR, I’d like to discuss this to determine whether the behaviour is intended.
Currently, onAfterChange
runs even when the route redirects. This can result in onAfterChange
running twice with the same result, as the middleware will have already changed the current page (via getState()
).
Some of this could be due to a race condition in my particular case (async functions), but it seems to be that the change should “resolve” prior to running a function like onAfterChange
.
Agree or disagree?
Issue Analytics
- State:
- Created 6 years ago
- Comments:9 (9 by maintainers)
Top Results From Across the Web
InfoPath 2010 The number of rule actions or the ... - MSDN
InfoPath 2010 The number of rule actions or the number of calls to the OnAfterChange event for a single update in the data...
Read more >Get MUI Slider value in onDragStop event
How can I get the current value of the Slider in the onDragStop function? Note, I'm unable to use this , as it...
Read more >React For Beginners - Building React Component - drag13.io
onAfterChange : (v: number) => void; ... it should log into the console next text: `redirecting user to the next {link}` where `{link}`...
Read more >Advanced C#Bot Development
... Name" model={this.region} modelProperty="regionName" onAfterChange={this. ... inside of a SecuredPage component, they are redirected to a 404 page.
Read more >how to use response.redirect in asp.net c#, Response redirect to ...
onafterchange angular · AsyncTask with progress dialog example in android · how to update edmx in visual studio 2017 · Passing data between...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
this is done. Additional info is in a comment to the PR:
https://github.com/faceyspacey/redux-first-router/pull/101
Been caught up with some client work, but I’ve decided I’m gonna go the mile on this one and all the related before change promises, redirects and route level versions of it. It’s just needed after all.