Skip to content

stop calling sound when preparing call#6103

Merged
mahibi merged 1 commit intomasterfrom
bugfix/noid/stopCallingSound
Apr 22, 2026
Merged

stop calling sound when preparing call#6103
mahibi merged 1 commit intomasterfrom
bugfix/noid/stopCallingSound

Conversation

@mahibi
Copy link
Copy Markdown
Collaborator

@mahibi mahibi commented Apr 22, 2026

Some users complain about ringtone continuing to ring when a call is joined. It could not be reproduced but code wise it makes sense to call stopCallingSound here.

This is adapted from the PR
#6092

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

Some users complain about ringtone continuing to ring when a call is joined.
It could not be reproduced but code wise it makes sense to call stopCallingSound here.

This is adapted from the PR
#6092

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@mahibi mahibi self-assigned this Apr 22, 2026
@mahibi mahibi added the 3. to review Waiting for reviews label Apr 22, 2026
@mahibi mahibi requested review from Copilot and sowjanyakch April 22, 2026 10:12
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Stops the calling/ringtone sound when a call is being prepared to prevent it from continuing after the user has joined a call.

Changes:

  • Call stopCallingSound() at the start of prepareCall().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

Codacy

Lint

TypemasterPR
Warnings8887
Errors00

SpotBugs

CategoryBaseNew
Bad practice66
Correctness1010
Dodgy code5353
Internationalization33
Malicious code vulnerability33
Performance44
Security11
Total8080

@mahibi mahibi merged commit 5361576 into master Apr 22, 2026
22 of 26 checks passed
@mahibi mahibi deleted the bugfix/noid/stopCallingSound branch April 22, 2026 15:29
@mahibi
Copy link
Copy Markdown
Collaborator Author

mahibi commented Apr 22, 2026

/backport to stable-23.0

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants