fridays / next-routes

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

Refactor use of Object.assign to spread operator. #54

Closed jaydenseric closed 7 years ago

jaydenseric commented 7 years ago

Fixes https://github.com/fridays/next-routes/issues/53.

Without understanding the source, I tried to faithfully recreate the logic using spread operator syntax. I am not sure how to test each of the functions that was changed, so I suggest looking over it carefully. It at least seems ok with pushRoute in my app, and IE 11 is now happy.

fridays commented 7 years ago

Thanks for contributing! I think we can fix this via babel. Let me check.

fridays commented 7 years ago

Please update, I changed the babel presets