Merge params and queryParams
guilhermeaiolfi opened this issue · comments
Guilherme Aiolfi commented
In the enter hood merge params like:
var params = jQuery.extend(ctx.transition.routes.queryParams, current_route.params);
jQuery is already available. But maybe lodash should be considered.
Andrew Smith commented
I saw this and was wondering why not use lodash _.deepClone
method instead? I might not have/want jQuery in my project at all.
Guilherme Aiolfi commented
Yes. Using _.deepClone
there and remove the jQuery.on/off`` for the
hashchange``` event we could remove the jQuery dep.