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 to keep session #1723

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

fix to keep session #1723

wants to merge 1 commit into from

Conversation

mahibi
Copy link
Collaborator

@mahibi mahibi commented Nov 22, 2021

WIP, will be improved.

when you were in a chat and received a call notification, then in onDetach of the chatview the session was removed by leaving the room. This fix will check if a call notification is ongoing and suppress to leave the room.

leaving the room in "onDetach" might cause more problems..(for example on first call when granting permissions?!)

TODO:

  • check which issues might be related to the onDetach& leaveRoom handling

Signed-off-by: Marcel Hibbe dev@mhibbe.de

@mahibi mahibi added this to the 13.0.0 milestone Nov 22, 2021
@mahibi mahibi marked this pull request as draft November 23, 2021 16:10
when you were in a chat and received a call notification, then in onDetach of the chatview the session was removed by leaving the room. This fix will check if a call notification is ongoing and suppress to leave the room.

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@mahibi mahibi force-pushed the bugfix/noid/fixToKeepSession branch from 03280b7 to 933a82f Compare November 23, 2021 19:44
@nextcloud-android-bot
Copy link
Collaborator

Lint

TypemasterPR
Warnings223224
Errors11

SpotBugs (new)

Warning Type Number
Bad practice Warnings 12
Correctness Warnings 121
Experimental Warnings 2
Internationalization Warnings 9
Malicious code vulnerability Warnings 168
Performance Warnings 29
Security Warnings 4
Dodgy code Warnings 214
Total 559

SpotBugs (master)

Warning Type Number
Bad practice Warnings 12
Correctness Warnings 121
Experimental Warnings 2
Internationalization Warnings 9
Malicious code vulnerability Warnings 168
Performance Warnings 29
Security Warnings 4
Dodgy code Warnings 214
Total 559

Lint increased!

@github-actions
Copy link
Contributor

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/1723-talk.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud Talk app.

@mahibi
Copy link
Collaborator Author

mahibi commented Nov 23, 2021

i think i was on the wrong track here.
While it would still make sense to keep the session for this case (might be improved), the true error seems to be #1725

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

Successfully merging this pull request may close these issues.

2 participants