Skip to content

Comments

[FIX] "Click to Join" button is not working if there are no muted users in the room#22871

Merged
tassoevan merged 4 commits intodevelopfrom
fix/click-to-join-muted
Aug 6, 2021
Merged

[FIX] "Click to Join" button is not working if there are no muted users in the room#22871
tassoevan merged 4 commits intodevelopfrom
fix/click-to-join-muted

Conversation

@matheusbsilva137
Copy link
Contributor

Proposed changes (including videos or screenshots)

  • Add check to room.muted array so as to cover the case in which it is undefined;

Issue(s)

Task - ClickUp
Fixes #22845

Steps to test or reproduce

Start a call in a room and, logged in as another user, press the "Click to Join!" button (system message) which has been sent in the room. No errors should be thrown in the console.

Further comments

@gabrieleiro gabrieleiro requested a review from a team August 5, 2021 20:12
@tassoevan tassoevan merged commit 14f8379 into develop Aug 6, 2021
@tassoevan tassoevan deleted the fix/click-to-join-muted branch August 6, 2021 12:19
tassoevan added a commit that referenced this pull request Aug 6, 2021
gabriellsh added a commit that referenced this pull request Aug 10, 2021
* 'develop' of github.com:RocketChat/Rocket.Chat:
  [FIX] Manually approve new users is not applied to SAML users (#22823)
  [FIX] applyChatRestictions callback not working for community version (#22839)
  [FIX] Message update not working in some cases (#22856)
  [FIX] Broken download link on uploaded files (#22848)
  [FIX] User is still asked for 2FA confirmation even if 2FA is globally deactivated. (#22801)
  [FIX] Registration not possible with TOTP and email verification (#22778)
  [FIX] users registered via third party apps bypass custom required fields (#22396)
  [FIX] "Click to Join" button is not working if there are no muted users in the room (#22871)
  [BREAK][ENTERPRISE] CSV file downloaded in the Engagement Dashboard's New Users section contains undefined data (#22862)
  Fix Local time conversion in the Users By Time of the Day chart (#22836)
@sampaiodiego sampaiodiego added this to the 3.17.1 milestone Aug 13, 2021
sampaiodiego pushed a commit that referenced this pull request Aug 16, 2021
…rs in the room (#22871)

* Fix room.muted undefined error

* Remove unnecessary includes function calls

Co-authored-by: Tasso Evangelista <tasso.evangelista@rocket.chat>
@sampaiodiego sampaiodiego mentioned this pull request Aug 16, 2021
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.

get error when click Video chat "click to join"

3 participants