fridays / next-routes

Universal dynamic routes for Next.js
MIT License
2.47k stars 230 forks source link

next-routes is not working in iOS 8.4 #32

Closed davidnguyen11 closed 7 years ago

davidnguyen11 commented 7 years ago

Hi!

Currently, I met some issues on iOS 8.4 devices.

When using <Link>

printWarning — commons.js:102Warning: unmountComponentAtNode(): The node you're attempting to unmount was rendered by another copy of React.

React DOM tree root should always have a node reference.

When using Router.pushRoute('index', params)

TypeError: undefined is not a function (evaluating '_this5.keyNames.includes(key)')
davidnguyen11 commented 7 years ago

I just added babel-polyfill & everything work fine. Could I update a document to mention about this issue?

fridays commented 7 years ago

Seems related to https://github.com/fridays/next-routes/issues/27 and https://github.com/zeit/next.js/pull/1808

davidnguyen11 commented 7 years ago

I have created 2 PR. You could check it out:

fridays commented 7 years ago

Thank you! It should be fixed with related problems by https://github.com/fridays/next-routes/pull/37. Please update to 1.0.25 and check again

HaNdTriX commented 7 years ago

Is this still an issue @nndung179?