Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

i love this mirotalk but there is some bugs #29

Closed
Nemra1 opened this issue Jun 7, 2021 · 15 comments
Closed

i love this mirotalk but there is some bugs #29

Nemra1 opened this issue Jun 7, 2021 · 15 comments
Labels
improvements Improvements question Further information is requested

Comments

@Nemra1
Copy link

Nemra1 commented Jun 7, 2021

by luck i found this project and am very happy to find it
Now my plan is integrate to new stream php script with mysql but i found some bugs..idont no if you know about it
1-switch camera in mobile
not working with error and after searching i found this test link and i guess have same problem too
https://webrtc.github.io/samples/src/content/devices/input-output/
2- clear chat is deleted locally only not for all users (im not sure if bug or not )
3- anyone can kick anyone even the visitors allowed to kick out the owner of room (im not sure if bug or not )
4- max 6 to 8 per room ( i know is peer to peer connection ) but if you just make any guide about integrate SFU to make it more wider for many users ..i will glad to help in this

and finally ...your project is saved my time and i am very very happy to find it after long search.. i meet mediasoup and others projects but its takes long time to study and modifications....

in the end you help me and thank you very much Mr @miroslavpejic85
And everyone working on this amazing project.. ❤️❤️❤️

@Jaideep25
Copy link
Contributor

Hey @Nemra1 Welcome! Glad you liked @miroslavpejic85 's project! Actually, clear chat, kicking anyone isn't a bug, it's made like that! And I guess many can join cuz @miroslavpejic85 had shown me a screenshot that more people can join!

@Nemra1
Copy link
Author

Nemra1 commented Jun 7, 2021

@Jaideep25
but switch cam still important problem.. and really need some hand to help about that

@Jaideep25
Copy link
Contributor

Actually the switch cam works for a few people.... so I don't think it's a bug!

@Nemra1
Copy link
Author

Nemra1 commented Jun 7, 2021

@Jaideep25
This is what im talk about
https://ibb.co/f4GQKQd

@Jaideep25
Copy link
Contributor

mm, may I know your OS in mobile, and which browser version?

@Nemra1
Copy link
Author

Nemra1 commented Jun 7, 2021

@Jaideep25
Huawei Y9 2020 with normally chrome browser from google play...also same problem for others browser like firefox
this issue in mobile only but is working on desktop good
what make me confused ... im already trying another projects with webrtc and switch camera is working good on mobile ... but im stuck on this..

@miroslavpejic85 miroslavpejic85 added the question Further information is requested label Jun 7, 2021
@miroslavpejic85
Copy link
Owner

miroslavpejic85 commented Jun 7, 2021

Hi @Nemra1, Thank you so much, i'm glad you like mirotalk and that you found it useful! ♥️
I'll answer you below, even if @Jaideep25 😇 something has already answered you.

  1. I need to reproduce it to try to fix. For now it had never happened to me. But there is always the first time 🙂
    Out of curiosity, if you press the swap camera button on mobile device, do you have the same problem?
    image
  2. Not a Bug, anyone clear his chat box conversation if want.
  3. Not a Bug anyone can feel free to kick the other if they want, maybe in the future I could put some limitations, in the sense that only the owner of the room can do certain actions.
  4. Yeah, Integrate SFU it would be very good goal! 💪

Mirotalk is not a finished project, it will be improved further and the contributors are always welcome! ♥️

@Nemra1
Copy link
Author

Nemra1 commented Jun 7, 2021

@miroslavpejic85 im glad to. Meet u sir..
1- yes same as for swipe button still showing same problem... So i have here - switch cam. In option panel and swipe button (same error) - android EMUi 9.1
About other os.. Tested on windows 10 with really camera device and virtual camera device.. That i was use it to replace my cam with some video.. Is working fine.. I

@miroslavpejic85
Copy link
Owner

miroslavpejic85 commented Jun 8, 2021

Hi @Nemra1, It could be an incompatibility issue (os - browser version) or something else, with the official API being used to gain access to the webcam. I try to reproduce it, as soon as I have time. 😉
Until now it has always worked well for both PC-MOBILE deivces.

@Nemra1
Copy link
Author

Nemra1 commented Jun 8, 2021

@miroslavpejic85 best luck to you.. I hope you going to bulid it with cordova.. Is much easier for mobile os

@miroslavpejic85
Copy link
Owner

miroslavpejic85 commented Jun 9, 2021

Hi @Nemra1.
Try now to swap the camera, I enabled also the error name to see what can be the problem in your case:
if you have the error like NotReadableError : Could not start video source
It most commonly happens when the browser tries to access the webcam or mic but they're already in use (by Skype, Facebook video chat ... etc.) and/or the browser can't fetch the device due to incompatibility issues.
it appears not to be a mirotalk bug. 😇
ref: https://blog.addpipe.com/common-getusermedia-errors/

@miroslavpejic85
Copy link
Owner

miroslavpejic85 commented Jun 13, 2021

@Nemra1 @Jaideep25
I think for this task, I can't do anything about it, it doesn't seem like a mirotalk issue. 👀

@Nemra1
Copy link
Author

Nemra1 commented Jun 13, 2021

@miroslavpejic85 i will try tonight.. Thank you

@miroslavpejic85
Copy link
Owner

miroslavpejic85 commented Jun 14, 2021

Ok I found what the problem can be in certain cases like yours.
It will be fixed asap 😉

@miroslavpejic85
Copy link
Owner

miroslavpejic85 commented Jun 14, 2021

Hi @Nemra1
Now you should be able to swap camera on any mobile devices. 🙂

@miroslavpejic85 miroslavpejic85 added bug Something isn't working improvements Improvements and removed bug Something isn't working labels Jun 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvements Improvements question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants