Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

switched out underscore with lodash #869

Merged
merged 1 commit into from
Jan 27, 2014
Merged

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Jan 25, 2014

lodash offers several additional function and many performance enhancements

spalger pushed a commit to spalger/kibana that referenced this pull request Jan 25, 2014
Create the store by passing in a scope, name, and default value. The current value is are fetched from localStorage, and changes to the object are synched with localStorage. The value is JSON stringified, so only store primative/simple values and the value must be an object.

Includes changes from elastic#869
@spalger spalger mentioned this pull request Jan 25, 2014
@rashidkpc
Copy link
Contributor

Have you tested out anything that uses the mixins? There isnt much and I don't see any reason for them to fail, I just want to be sure here.

@spalger
Copy link
Contributor Author

spalger commented Jan 25, 2014

Not really, no. Will do

@spalger spalger mentioned this pull request Jan 26, 2014
spalger pushed a commit that referenced this pull request Jan 27, 2014
switched out underscore with lodash
@spalger spalger merged commit 2c19a45 into elastic:master Jan 27, 2014
@spalger spalger deleted the lodash branch January 27, 2014 15:50
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