theonion / django-bulbs

DEPRECATED: This project is now part of the Mono Repo (https://github.com/theonion/omni)
MIT License
26 stars 7 forks source link

Parse twitter video #201

Closed MichaelButkovic closed 8 years ago

MichaelButkovic commented 8 years ago

@mparent61 @spra85 @benghaziboy This will parse the twitter videos that are currently blowing up the star wipe feed

spra85 commented 8 years ago

@MichaelButkovic, are we missing a piece in the renderer that renders the expected IA format?

mparent61 commented 8 years ago

LGTM

mparent61 commented 8 years ago

@MichaelButkovic -- Let's package this fix with this release too: https://github.com/theonion/django-bulbs/pull/202

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.02%) to 85.8% when pulling 858c85990d3ce8027947d0aee114a786bf841352 on ia-twitter-video into 941d6dd480974a452867fbf3ff5bca2cc389eef5 on master.

MichaelButkovic commented 8 years ago

@mparent61 pulled in your changes here

mparent61 commented 8 years ago

LGTM

spra85 commented 8 years ago

Same, 👍

coveralls commented 8 years ago

Coverage Status

Changes Unknown when pulling 5bffa5030655c59b20348dd92e18d76b39817d3f on ia-twitter-video into \ on master**.