search

citcit / autoembed

Automatically exported from code.google.com/p/autoembed
GNU Lesser General Public License v3.0
0 stars 0 forks source link

Erroneous site params #1

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
To slim down this beast, I think there are few more questionable params 
that we (in my opinion) don't need.

1. 'plugin' - I think the goal for this project should be focused on flash 
only, so plugin isn't needed.
2. 'type' - 98% are type "video", and there are a couple with type "pop" 
and "audio".  I'm not entirely sure what "pop" type is - it's just video 
right?  Also, type "audio" isn't important to support IMO.
3. 'website' and 'title' are harmless cruft, and can stay imo.

Thoughts?

Original issue reported on code.google.com by jhi...@gmail.com on 31 Dec 2008 at 8:47

GoogleCodeExporter commented 8 years ago 
Thinking more on this, supporting a type 'audio' may actually be a good idea. 
There 
are tons of music sites (last.fm, myspace, etc) and since we already have the 
infrastructure to support it, we might as well keep it.

Closing this ticket.

Original comment by jhi...@gmail.com on 31 Dec 2008 at 10:54