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

Sheets: Make "batchGet"/range option of type string[] instead of string #1239

Conversation

arthurjdam
Copy link

@arthurjdam arthurjdam commented Jul 17, 2018

Not sure why the documentation doesn't agree, but there does not seem to be any way of sending a singular string as batchGet's "ranges" option
Example:
ranges: "TABONE!A1:B,TABTWO!A2:B" will fail
ranges: ["TABONE!A1:B", "TABTWO!A2:B"] works just fine

  • npm test succeeds
  • Pull request has been squashed into 1 commit
  • I did NOT manually make changes to anything in apis/
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate JSDoc comments were updated in source code (if applicable)
  • Appropriate changes to readme/docs are included in PR

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 17, 2018
@JustinBeckwith
Copy link
Contributor

Thank you so much for the PR! Sadly, this is an auto-generated file. Any fixes required will need to be made on the generator. We're tracking this over in #1266 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants