search

collective / wildcard.media

13 stars 12 forks source link

Youtube iframe #36

Closed cekk closed 8 years ago

cekk commented 8 years ago

Hi, this pr is related to the issue #29. Now the youtube_url input can take some different types of url and in the view will be used the iframe to embed the video.

I've made some basic tests. Let me know if you want more complex tests.

There is an old test that fails..maybe something is changed and now there is only 1 tag in the html?

vangheem commented 8 years ago

Looks good. It'd be great to fix that failing test as well but I'll merge this as is.