Seq wrapper #270

Open
wants to merge 5 commits into
from

Conversation

Projects
None yet
2 participants
@ScottDowne
Contributor

ScottDowne commented Jan 21, 2013

No description provided.

@@ -366,6 +366,7 @@
// See if we can use a wrapper directly, if not, try players.
for ( j = 0; j < wrappers.length; j++ ) {
mediaWrapper = Popcorn[ wrappers[ j ] ];
+ srci = srci.replace( /^https\:\/\/soundcloud\.com/, "http://soundcloud.com" );

This comment has been minimized.

Show comment Hide comment
@rwaldron

rwaldron Jan 21, 2013

Contributor

This comment has been minimized.

Show comment Hide comment
@ScottDowne

ScottDowne Jan 21, 2013

Contributor

Totally just hacking up something for debugging purposes.

Not in review yet :)

Not yet sure how to solve it, though.

@ScottDowne

ScottDowne Jan 21, 2013

Contributor

Totally just hacking up something for debugging purposes.

Not in review yet :)

Not yet sure how to solve it, though.

This comment has been minimized.

Show comment Hide comment
@rwaldron

rwaldron Jan 21, 2013

Contributor

How to solve this particular part or the whole thing?

@rwaldron

rwaldron Jan 21, 2013

Contributor

How to solve this particular part or the whole thing?

This comment has been minimized.

Show comment Hide comment
@ScottDowne

ScottDowne Jan 21, 2013

Contributor

soundcloud seems to be breaking on https. not sure why. Have not had time to dig into it or move this hack to a better spot yet.

@ScottDowne

ScottDowne Jan 21, 2013

Contributor

soundcloud seems to be breaking on https. not sure why. Have not had time to dig into it or move this hack to a better spot yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment