Alex, in a pull request as large as this, please respond to comments with a
description of what you did.  This description "Fixed." doesn't tell me
what you changed, and Github doesn't let me see the comment I originally
made from this one.


On Tue, Oct 15, 2013 at 6:44 AM, Alexander Kryklia <notifications@github.com
> wrote:

> In cms/djangoapps/contentstore/transcripts_utils.py:
>
> > +
> > +def get_transcripts_from_youtube(youtube_id):
> > +    """
> > +    Gets transcripts from youtube for youtube_id.
> > +
> > +    Returns (status, transcripts): bool, dict.
> > +    """
> > +    html_parser = HTMLParser.HTMLParser()
> > +    utf8_parser = etree.XMLParser(encoding='utf-8')
> > +    settings.YOUTUBE_API['params']
> ['v'] = youtube_id
> > +    data = requests.get(
> > +        settings.YOUTUBE_API['url'],
> > +        params=settings.YOUTUBE_API['params']
> > +    )
> > +    if data.status_code != 200 or not data.text:
> > +        log.debug("Can't receive correct transcripts from Youtube.")
>
> Fixed.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/1111/files#r6967701>
> .
>