GoogleWebComponents / google-youtube-upload

Element enabling you to upload videos to YouTube.
Other
40 stars 28 forks source link

Initial 0.8 port #7

Closed atotic closed 9 years ago

atotic commented 9 years ago

Not working due to gapi errors. 0.5 component has the same error. this.accessToken = e.detail.result.access_token;

ebidel commented 9 years ago

@jeffposnick can you take a look?

jeffposnick commented 9 years ago

Using the gapi.auth2 library to authenticate with YouTube-related scopes isn't viable right now, due to the +Page issue brought up in https://github.com/GoogleWebComponents/google-youtube-upload/pull/5 (and tracked via an internal bug as well).

I can do a review and exercise the code while logging in as a real Google Account, but I don't think we can consider this element ready for general consumption until it depends on a version of <google-signin> that allows you to auth as a +Page.

atotic commented 9 years ago

I was not aware of gapi.auth2 problem. But demo does work on my local machine, using my atotic@gmail.com account.

jeffposnick commented 9 years ago

The ported code all seems to work well when I try it with a non +Page YouTube account, so :+1: in general to this PR, and thanks! The changes that need to be made to support +Page accounts are in the backend of Google Signin 2.0, so I doubt anything will need to change in this PR before it's merged.

But like I said, I'm hesitant to make much noise about this element (e.g., via the YouTube API social channels) until it gets fixed.

atotic commented 9 years ago

I am fixing this element for 0.9, new changes coming in soon

On Fri, May 8, 2015 at 10:54 AM, Jeffrey Posnick notifications@github.com wrote:

The ported code all seems to work well when I try it with a non +Page YouTube account, so [image: :+1:] in general to this PR, and thanks! The changes that need to be made to support +Page accounts are in the backend of Google Signin 2.0, so I doubt anything will need to change in this PR before it's merged.

But like I said, I'm hesitant to make much noise about this element (e.g., via the YouTube API social channels) until it gets fixed.

— Reply to this email directly or view it on GitHub https://github.com/GoogleWebComponents/google-youtube-upload/pull/7#issuecomment-100311771 .

atotic commented 9 years ago

Works

jeffposnick commented 9 years ago

:+1:

addyosmani commented 9 years ago

LGTM2

ebidel commented 9 years ago

LGTM @jeffposnick you get the final say.