Zeno97 / cordova-plugin-jitsi-meet-sdk

Cordova plugin for Jitsi Meet React Native SDK.
Apache License 2.0
8 stars 11 forks source link

`JitsiMeet.disposeConference()` is not working #17

Closed youssmak closed 2 years ago

youssmak commented 2 years ago

I'm reproducing on Android 11 or Android 9

error :

JitsiMeet not started

even if a conference is currently running

Zeno97 commented 2 years ago

I think this is related with the hangup button issue. I will test with android 11 and let you know asap

Inviato dal mio Galaxy

-------- Messaggio originale -------- Da: Youssef Makboul @.> Data: 19/11/21 18:25 (GMT+01:00) A: Zeno97/cordova-plugin-jitsi-meet-sdk @.> Cc: Subscribed @.***> Oggetto: [Zeno97/cordova-plugin-jitsi-meet-sdk] JitsiMeet.disposeConference() is not working (Issue #17)

I'm reproducing on Android 11 or Android 9

error :

JitsiMeet not started

even if a conference is currently running

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/Zeno97/cordova-plugin-jitsi-meet-sdk/issues/17, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKABCKDDRJPPPSJ7BJBGIQDUM2CBNANCNFSM5IMUM2TA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Zeno97 commented 2 years ago

Hi it should be solved. I've fired hangup event instead of directly killing activity. I hope also solves #16 Please test it and let me know ASAP

youssmak commented 2 years ago

Hi @Zeno97, thks for the quick fix

Now the FATAL ERROR occurs before the success / error callback of JitsiMeet.disposeConference() is fired

The application Crash with the same error as when clicking on hang up button https://github.com/Zeno97/cordova-plugin-jitsi-meet-sdk/issues/16

Zeno97 commented 2 years ago

Solved in #16