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

Regression: Prevent connect to asterisk when VoIP is disabled #24601

Merged
merged 8 commits into from
Feb 25, 2022

Conversation

murtaza98
Copy link
Contributor

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@murtaza98 murtaza98 requested review from a team and rodrigok February 24, 2022 08:00
@murtaza98 murtaza98 marked this pull request as ready for review February 24, 2022 08:01
@renatobecker renatobecker added this to the 4.5.0 milestone Feb 24, 2022
@murtaza98 murtaza98 force-pushed the voip/prevent-voip-service-if-disabled branch from adefe98 to 3d42474 Compare February 25, 2022 11:02
Co-authored-by: Kevin Aleman <kevin.aleman@rocket.chat>
@murtaza98
Copy link
Contributor Author

@sampaiodiego adding you as a reviewer over here since I wanted to confirm if it is safe to call settings.watch within a service. Please advice 🙏 Thanks in advance

@KevLehman KevLehman changed the title Regression: Prevent trying to connect to asterisk when VoIP is disabled Regression: Prevent connect to asterisk when VoIP is disabled Feb 25, 2022
@KevLehman KevLehman merged commit a88a96f into develop Feb 25, 2022
@KevLehman KevLehman deleted the voip/prevent-voip-service-if-disabled branch February 25, 2022 20:17
gabriellsh added a commit that referenced this pull request Feb 28, 2022
…age-template-2

* 'develop' of github.com:RocketChat/Rocket.Chat: (61 commits)
  Regression: Server crashing if Voip credentials are invalid (#24646)
  Regression: Extension List panel UI not aligned with designs (#24645)
  Regression: Queue counter aggregator for incoming/hanged calls (#24635)
  Regression: Fix double value on holdTime and empty msg on last message (#24630)
  Regression: If Asterisk suddenly goes down, server has no way to know. Causes server to get stuck. Needs restart (#24624)
  Regression: Prevent connect to asterisk when VoIP is disabled (#24601)
  Regression: Encode registration info as JWT when signing key is provided (#24626)
  Regression: Fix time fields and wrap up in Voip Room Contexual bar (#24625)
  Regression: Fix in-correct room status shown to agents (#24592)
  Regression: Do not show toast on incoming voip calls (#24619)
  Regression: Fix incoming voip call ringtone is not ringing (#24616)
  Regression: Mark all rooms as read modal closing instantly. (#24610)
  Regression: Fix translations for call started message (#24615)
  Regression: Bunch of settings fixes for VoIP (#24594)
  Regression: Admin Sidebar colors inverted (#24609)
  Regression: No audio when call comes from Skype/IP phone (#24602)
  Regression: Fixes in Voice Contextual Bar and Directory (#24596)
  Regression: Fix time format on Voip system messages (#24603)
  Regression: VoIP service button displayed when VoIP is disabled (#24598)
  Add support to namespace within micro services (#24581)
  ...
@pierre-lehnen-rc pierre-lehnen-rc mentioned this pull request Mar 1, 2022
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