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

Fix VoiceService playing when app is terminated #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix VoiceService playing when app is terminated #23

wants to merge 2 commits into from

Conversation

Frezyx
Copy link

@Frezyx Frezyx commented Feb 20, 2021

Привет @grishka ✋🏻

При использовании приложения заметил баг.
Когда открываешь комнату, выходишь полностью из приложения и убираешь из запущенных в фоне - трансляция всё равно идет без возможности прекратить это.

Я добавил код, который прекращает работу сервиса в том случае, если приложение полностью выключено (terminated).

@LionZXY
Copy link

LionZXY commented Feb 20, 2021

Так это же фича, так и работает iOS приложение

@staser
Copy link

staser commented Feb 20, 2021

Возможно, было бы лучше добавить кнопку в нотификацию для остановки.

Также сделано у MS Teams.

@LionZXY
Copy link

LionZXY commented Feb 20, 2021

Foreground service?

@Frezyx
Copy link
Author

Frezyx commented Feb 20, 2021

Так это же фича, так и работает iOS приложение

Не, когда приложение просто скрыто, это норма. Но, когда приложение из запущенных убираешь, а чат продолжается - вроде это не особо круто.

@grishka
Copy link
Owner

grishka commented Feb 20, 2021

Ну вообще это больше фича, чем баг, да. Все voip-звонки и музыкальные плееры в андроиде так работают.

@staser
Copy link

staser commented Feb 20, 2021

Вот да, думаю, что оптимально - добавить фикс из PR.

Но и в кнопку добавить выход из чата.

@Frezyx
Copy link
Author

Frezyx commented Feb 20, 2021

Ну вообще это больше фича, чем баг, да. Все voip-звонки и музыкальные плееры в андроиде так работают.

Да, я полностью согласен и сервис не убивается (с моими изменениями), когда приложение висит в скрытых. Но работа сервиса прекращается, когда приложение полностью убито.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants