Update to react-router@2

[bguiz]

Initially left a comment here, but feel more appropriate to raise an issue here.

@doctyper It seems that several of us here are having issues. Just
confirming that you are using it for server rendering as well?

If so, could you please update the react-helmet-example to use
react-router@2 (it's currently using [email protected]):
https://github.com/mattdennewitz/react-helmet-example/blob/master/package.json#L12

... and then I'd like to see what is done differently when rendering
on the server: https://github.com/mattdennewitz/react-helmet-example/blob/master/server.js#L30-L48

CC @mattdennewitz

[aybmab]

Any updates?

[mattdennewitz]

Sure, will take this on shortly.

[asiniy]

?

[vignestion]

Well take a look at this. @aybmab @bguiz
React-helmet example app

[aybmab]

Not sure if they fixed it, but my team found a memory leak using react helmet. I ended up just building my own custom solution, which is actually pretty easy/straight-forward.

[doctyper]

@aybmab Interesting. Please open a ticket in react-helmet if you're still experiencing this leak.

[mattdennewitz]

Anyone interested in implementing React Router 3 or 4?