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

Update sheets snippets to use the latest Node client library version #47

Open
YOEL311 opened this issue Dec 19, 2018 · 6 comments
Open
Labels
help wanted Extra attention is needed

Comments

@YOEL311
Copy link

YOEL311 commented Dec 19, 2018

  • Node version (node -v)8
  • OS (Mac/Linux/Windows)heroku

In this part

console.log(`${result.updates.updatedCells} cells appended.`);

There is a mistake
He gives Jason a circular
I opened it by

     var CircularJSON = require('circular-json');
     console.log(CircularJSON.stringify(result));

That's the only way it works for me now

     console.log(JSON.stringify(result.data.updates));
@erickoledadevrel erickoledadevrel self-assigned this Dec 21, 2018
@erickoledadevrel erickoledadevrel changed the title Wrong path Update sheets snippets to use the latest Node client library version Dec 21, 2018
@erickoledadevrel
Copy link

The problem is that the sheets snippets are written against an older version of the Node.js library, which didn't use the data field. I've updated the issue accordingly.

@YOEL311
Copy link
Author

YOEL311 commented Dec 23, 2018

Older than NODE 8?
Today there is already NODE 11
  The world is progressing
  Do not stay behind

@erickoledadevrel
Copy link

An older version of the googleapis dependency is what I should have said.

@erickoledadevrel erickoledadevrel removed their assignment Jan 11, 2019
@erickoledadevrel erickoledadevrel added the help wanted Extra attention is needed label Jan 11, 2019
@fhinkel
Copy link

fhinkel commented Mar 15, 2019

Hi, do you know for which method this snippet is or where to find it in the docs? Thanks.

@anantakrishna
Copy link

https://github.com/gsuitedevs/node-samples/blob/master/sheets/snippets/snippets.js#L213

Here also seems outdated, currently the property is called requestBody.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants