Youku

[NotCJSBirdie]

I would like to see the source code and contribute

[cookpete]

I forgot that branch existed! I just need someone with a Youku client id to test that it works.

@NotCJSBirdie @ImranBugCMG @rocklaam are you able to test the youku branch?

Something like:

<ReactPlayer
  url='https://v.youku.com/v_show/id_XNTg4NjExNzY3Ng==.html'
  config={{ clientId: '12345' }}
/>

[ImranBugCMG]

@cookpete so far findings are that example document is not working for youku in this branch.

In example code after entering youku url in input field, it shows an error using the alert method of browser which reads: [Fail]The params of {vid,target,client_id} are necessary !
Note: clientId was provided.

and in console I was getting following error: TypeError: Cannot read properties of null (reading 'getElementsByTagName')

If I get time I will try to debug this.

[cookpete]

Thanks for trying @ImranBugCMG – If you are happy to privately share your client ID with me, I can try and debug it too. I’m at https://twitter.com/cookpete or email pete [at] cookpete [dot] com

[ImranBugCMG]

Thanks for trying @ImranBugCMG – If you are happy to privately share your client ID with me, I can try and debug it too. I’m at https://twitter.com/cookpete or email pete [at] cookpete [dot] com

I have sent you an email

[AntonOfTheWoods]

Did any progress get made on this? I will have an ID to test with if you need one, and will have a look myself to try and get it working.

[FuYaoDe]

The variable name playerID here is incorrect, which leads to an error in referencing this.playerId below. Therefore, the error messages [Fail]The params of {vid,target,client_id} are necessary! and TypeError: Cannot read properties of null (reading 'getElementsByTagName') occur.

By changing it to playerId the issue can be resolved.