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

Start recording from Talk Android breaks participant list layout in the app #4635

Open
pmarini-nc opened this issue Jan 20, 2025 · 2 comments
Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) bug Something isn't working

Comments

@pmarini-nc
Copy link

pmarini-nc commented Jan 20, 2025

Steps to reproduce

  1. Start a call from Talk Android
  2. Join a call from a browser as a guest
  3. Start the recording from Talk Android

Expected behaviour

When starting the recording from the Talk Android the list of participants is broken. In an audio call with two persons the initial of the guest already in the call is not shown anymore and another participant is added (I guess the virtual participant for the recording). If the recording starter leaves and reenters the call the layout is back as normal.

Image
Image

Actual behaviour

See above

Device brand and model

Fairphone 4

Android version

14

Nextcloud Talk app version

20.1.1

Nextcloud server version

30.0.5

Talk version

20.1.3

Custom Signaling server configured

Yes (specify version in Additional Information)

Custom TURN server configured

Yes

Custom STUN server configured

Yes

Android logs

Server log

Additional information

  • signaling server: 2.0.1, compiled from source
  • recording server: 0.1
@pmarini-nc pmarini-nc added 0. Needs triage bug Something isn't working labels Jan 20, 2025
@mahibi
Copy link
Collaborator

mahibi commented Jan 24, 2025

iirc correctly i had a conversation about this with @danxuliu when call recording was introduced back then?

@mahibi mahibi added 1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) and removed 0. Needs triage labels Jan 24, 2025
@danxuliu
Copy link
Member

That is unexpected, as a display item should not be added for internal participants (like the recording server). But maybe there is a race condition between adding the participant and setting the internal flag 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants