Skip to content

Conversation

daigo424
Copy link

@daigo424 daigo424 commented May 5, 2018

Fixes: #606

Google::Apis::Core::HttpCommand.body was nil.

before:

Upload complete
Sending upload start command to https://www.googleapis.com/upload/youtube/v3/thumbnails/set?
Upload status final
Error - #<Google::Apis::ClientError: required: Required parameter: videoId>

after:

Sending upload command to https://www.googleapis.com/upload/youtube/v3/thumbnails/set?videoId=55XDuvqq1wI&upload_id=AEnB2UqGyNrZjrGaJYvfYEi3POef6dzvNqjYwSAH7zgYRmBa8Uvqkx6nq3g1h2SWvUq9ef8eq4cFj5gIZXMX7WkGe_qE9CxKFA&upload_protocol=resumable
Upload status final
Success - #<Google::Apis::YoutubeV3::SetThumbnailResponse:0x00007f90a0f4f320

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label May 5, 2018
@daigo424 daigo424 force-pushed the add_class_set_thumbnail_request branch from 8e2064c to 13cac89 Compare May 5, 2018 11:50
@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels May 5, 2018
@coveralls
Copy link

coveralls commented May 5, 2018

Coverage Status

Coverage remained the same at 95.997% when pulling 13cac89 on daigoishii:add_class_set_thumbnail_request into 1210eab on google:master.

@meanphil
Copy link

meanphil commented Oct 2, 2018

Can someone please review and merge this?

@hoangzinh
Copy link

Please review this PR

@dazuma
Copy link
Contributor

dazuma commented Dec 4, 2018

Unfortunately, this is automatically generated code. If we merge this, the automation will revert it the next time it runs. I'm investigating why the code generation is omitting this...

@JustinBeckwith JustinBeckwith added the 🚨 This issue needs some love. label Feb 7, 2019
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Feb 7, 2019
@grant
Copy link
Contributor

grant commented Jun 4, 2019

Unfortunately, this is automatically generated code. If we merge this, the automation will revert it the next time it runs. I'm investigating why the code generation is omitting this...

As mentioned here, we can't merge this PR. Continuing tracking the issue with #606.

@grant grant closed this Jun 4, 2019
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. 🚨 This issue needs some love.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

YouTube set_thumbnail not working

9 participants