Remove lodash

[simplenotezy]

I noticed you're pulling in lodash, but since you're using babel, why not stick to ES6?

For instance:

getPosts(limit, cb) {
  if (_.isEmpty(limit)) {
    let limit = 5;
  }

  axios
    .get(SETTINGS.API_BASE_PATH + "posts?per_page=" + limit)
    .then(response => {
      cb(response.data);
    })
    .catch(e => {
      cb(e);
    });
}

Could simply be:

getPosts(limit = 5, cb) {
  axios
    .get(SETTINGS.API_BASE_PATH + "posts?per_page=" + limit)
    .then({data} => {
      cb(data);
    })
    .catch(e => {
      cb(e);
    });
}

It would reduce filesize ;)

[simplenotezy]

Speaking of which; I don't see where / how loadash is being included - not present in the package.json or webpack file.

[simplenotezy]

Never mind; it was in package.json =)

[EvanAgee]

@simplenotezy yes this is on my list but I haven't had time yet. Interested in creating a PR to remove the dependency?

[simplenotezy]

Yes, I'll have a look at it @EvanAgee

[simplenotezy]

Just to have an idea of how much lodash takes:

[EvanAgee]

Yes, it's a big boy. I'll implement the tree shaking so at least it isn't larger than necessary.

[simplenotezy]

It's used sooo little it shouldn't take long time just to completely remove