Skip to content

Use lodash instead of underscore#1268

Merged
Kmaschta merged 4 commits intomasterfrom
Use-Lodash-instead-of-underscore
Dec 9, 2016
Merged

Use lodash instead of underscore#1268
Kmaschta merged 4 commits intomasterfrom
Use-Lodash-instead-of-underscore

Conversation

@Phocea
Copy link
Copy Markdown
Contributor

@Phocea Phocea commented Dec 8, 2016

As discussed with @fzaninotto, reopening this PR. I have left the original implementation with a global delcared in the vendor.js since this is the solution I tested.

On latest build I am getting the error _.includes is not a function when navigating into one of my custom pages.
Following several threads on Restangular Git (one of them being mgonto/restangular#1225). I found out that Restangular is now compatible with lodash 4.

ng-admin using underscore is causing incompatibilty since the new webpack dependency version has been merged. Making use of lodash fixes the problems

@Phocea
Copy link
Copy Markdown
Contributor Author

Phocea commented Dec 8, 2016

Rebase and replace #1252

@Phocea
Copy link
Copy Markdown
Contributor Author

Phocea commented Dec 8, 2016

Fix #1209

@Kmaschta
Copy link
Copy Markdown
Contributor

Kmaschta commented Dec 9, 2016

If you already have lodash, why do you need to add lodash.debounce in dependencies?

And don't you need lodash in dependencies instead of devDeps?

@Phocea
Copy link
Copy Markdown
Contributor Author

Phocea commented Dec 9, 2016

lodash doesnt include lodash.debounce by default.
lodash is needed by Restangular so not only in dev.

@Kmaschta Kmaschta merged commit db3d9dd into master Dec 9, 2016
@Kmaschta Kmaschta deleted the Use-Lodash-instead-of-underscore branch December 9, 2016 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants