Hey dude, sorry for the late reply this is my work email and I don't get it
to my phone.

The reason I did the isEmpty is that the spreadsheet data that I required
gets returned regardless of the type and I found that if I supplied a
spreadsheet id which I have been doing then it never got past this if
statement:

if(this[id])
    return callback(null);

I was not too sure about the meta as an option I thought it might be a bit
too much information for people wanting a basic usage and it could be a
little confusing, it's completely your call though.

Thank you very much for the feedback, I did try to keep the formatting the
same as yours, sorry I missed the camelCase.

Simon.


On Fri, Nov 15, 2013 at 11:42 AM, Jaime Pillora <notifications@github.com>wrote:

> In lib/index.js:
>
> >      if(err) return callback(err, null);
> >
> >      var result = JSON.parse(body);
> >      var entries = result.feed.entry || [];
> >
> > +    if(entries.length > 0 && _.isEmpty(_this.meta)) {
>
> instead of isEmpty what about type === "work"? maybe i needed to comment
> this function, though its used fetch all of the spreadsheets, then with the
> chosen spreadsheet id, its used again to fetch all the worksheets, where
> type is "sheet" or "work"
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jpillora/node-edit-google-spreadsheet/pull/14/files#r7687706>
> .
>