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

Participants count in call inaccurate #10677

Closed
gerbrent opened this issue Oct 10, 2023 · 1 comment · Fixed by #10843
Closed

Participants count in call inaccurate #10677

gerbrent opened this issue Oct 10, 2023 · 1 comment · Fixed by #10843
Milestone

Comments

@gerbrent
Copy link

gerbrent commented Oct 10, 2023

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. be in a call with many participants
  2. note the total participants number active in the call is incorrect

Expected behaviour

Call participant count should reflect actual participant count

Actual behaviour

Here, the participant count (top bar, beside call duration counter) is 3, but clearly there are more participants

273933036-014a6141-267a-4a0e-8367-eb29750e4542

Talk app

Talk app version: (see apps admin page: /index.php/settings/apps)

Custom Signaling server configured: yes/no and version (see additional admin settings: /index.php/index.php/settings/admin/talk#signaling_server)

Custom TURN server configured: yes/no (see additional admin settings: /index.php/settings/admin/talk#turn_server)

Custom STUN server configured: yes/no (see additional admin settings: /index.php/settings/admin/talk#stun_server)

Browser

Microphone available: yes/no
no

Camera available: yes/no
no

Operating system: Windows/Ubuntu/...
Linux/openSUSE Tumbleweed

Browser name: Firefox/Chrome/...
Firefox

Browser version: 85/96/...
118.01

Browser log

``` Insert your browser log here, this could for example include: a) The javascript console log b) The network log c) ... ```

Server configuration

cloud.nextcloud.com

Operating system: Ubuntu/RedHat/...

Web server: Apache/Nginx

Database: MySQL/Maria/SQLite/PostgreSQL

PHP version: 8.0/8.1/8.2

Nextcloud Version: (see admin page)

List of activated apps:

If you have access to your command line run e.g.:
sudo -u www-data php occ app:list
from within your server installation folder

Nextcloud configuration:

If you have access to your command line run e.g.:
sudo -u www-data php occ config:list system
from within your Nextcloud installation folder

Server log (data/nextcloud.log)

Insert your server log here
@Antreesy
Copy link
Contributor

getParticipants() mixin depends on isActive prop, and will update participants list only when true.
Should be the reason why it's not happening for TopBar and open RightSidebar with ParticipantsTab as inactive tab

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

Successfully merging a pull request may close this issue.

3 participants