search

callstack / react-native-opentok

React Native OpenTok
202 stars 79 forks source link

Support more then one Subscriber #51

Open philolo1 opened 7 years ago

philolo1 commented 7 years ago

I dont think you support yet more then one subscriber, i need this feature and will do a pull request for this. I think for this purpose the subscriberview should get a streamId property.

loctrandn commented 7 years ago

+1 I need support multi subscriber

dratwas commented 7 years ago

@philolo1 What you mean about multiple subscribers? This library supports multiple session so you can connect with two sessions and then render two subscribers view with different sessionId prop.

Are you using few streams in one session?

RoyalDeveloper2015 commented 7 years ago

can I test Publisher & Subscriber in one Component at the same time with . one Session ID ?

dratwas commented 7 years ago

Nope, because we're connecting with a session using token which is related to subscriber or publisher. I'm thinking if we can change our implementation, but Is it needed for you or it is just your test case?

RoyalDeveloper2015 commented 7 years ago

I'd like to build app similar to skype , there you can see you your video and others video . how can we implement like that using this SDK ?

dratwas commented 7 years ago

I'm not sure if it is possible. I have to check. At this moment you can't achieve that with this library.

philolo1 commented 7 years ago

its possible, you just need to use the streams instead of the sessions, i forked the library and made it worked, but my solution is not to good to merge it or do a pull request.

2017-10-31 19:09 GMT+09:00 Drapich Piotr notifications@github.com:

I'm not sure if it is possible. I have to check. At this moment you can't achieve that with this library.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/callstack/react-native-opentok/issues/51#issuecomment-340716579, or mute the thread https://github.com/notifications/unsubscribe-auth/AF_F5u4T9ZY2nNdd0qKgkm98TFWQmt2iks5sxvHMgaJpZM4QJWWk .

dratwas commented 7 years ago

Ok, so I think we will implement this feature in next releases :)

alexandermtang commented 6 years ago

@philolo1 would you be able to share your fork that uses streams instead of sessions?

siliconavengers commented 5 years ago

@philolo1 can you share your fork? We're facing the same issue, when we want to subscribe multiple streams