Replace cloneDeep with immer

ExtensionEngine / tailor

Content authoring platform

MIT License

31 stars 10 forks source link

Replace cloneDeep with immer #719

Closed ikovac closed 3 years ago

ikovac commented 3 years ago

Improve performance by replacing lodash/cloneDeep with immer

zcuric commented 3 years ago

Just a thought. Do we need a new utility library in the codebase? 🤔

MiroDojkic commented 3 years ago

@zcuric I share your concern, even though it has potential benefits of reducing scripting time and re-renders. We tried to simplify the introduced API, but it still seems too obtrusive for an unnoticeable gain, so I'm closing this one.