Commit ee7994c149d5a44c1ff65daf5c634777a9d139ee is broken on android

OpenVidu / openvidu-tutorials

OpenVidu tutorials to get started

http://openvidu.io/tutorials

Apache License 2.0

228 stars 255 forks source link

Commit ee7994c149d5a44c1ff65daf5c634777a9d139ee is broken on android #150

Closed jaswant0605 closed 3 years ago

jaswant0605 commented 3 years ago

The commit ee7994c149d5a44c1ff65daf5c634777a9d139ee is not working, Remote user's video is not visible. onAddTrack is never called. Use the commit from 13 Nov instead (Commit a24ecac4e478578ff94e5ac786eae3547285fc22), its working even on the latest instance (2.17.0 as of now).

pabloFuente commented 3 years ago

You are right. This commit is supposed to work with master version of OpenVidu Server. So the tutorial instructions (which directly clone master of openvidu-tutorials repository and use 2.17.0 release of openvidu-server) are actually broken right now for the openvidu-android tutorial.

git checkout 0752d4edcccc979d8c62b9b9fe4b86f70d6d7568 on openvidu-tutorials repository should avoid the problem. We will add "checkout" commands to the tutorials instructions to avoid this issue in the future.

vjvk commented 3 years ago

Hi, Please suggest me to checkout the exact tutorial for an android, latest commit which is not helping us. As I am unable to load remote participant's video. Also I tried this git checkout 0752d4edcccc979d8c62b9b9fe4b86f70d6d7568 commit, still I am facing the same issue. Web and iOS has no issues on connecting the videos. But in Android we facing an issue for past two days, unable to resolve. Kindly help us to figure out this as soon as possible, we are in urgent to release.

micaelgallego commented 3 years ago

Please use the updated instructions to clone the repository pointing to brach v2.17.0