[PR #36] [MERGED] fix: error when playing notification sounds in quick succession #35

Closed
opened 2026-09-04 00:49:48 +02:00 by Stone_Red · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/RedWizardsLab/EchoHub/pull/36
Author: @Stone-Red-Code
Created: 2/25/2026
Status: Merged
Merged: 4/20/2026
Merged by: @HueByte

Base: devHead: dev_fix_notification_sound


📝 Commits (2)

  • 802db1c fix: error when playing notification sounds in quick succession
  • ca97254 chore: add notification bug fix to changelog

📊 Changes

2 files changed (+21 additions, -3 deletions)

View changed files

📝 docs/changelog/v0.2.10.md (+1 -0)
📝 src/EchoHub.Client/Services/NotificationSoundService.cs (+20 -3)

📄 Description

Summary

Fixes errors when playing notification sounds in quick succession.

This fix may cause notification sounds to be delayed if the previous sound is still playing, but it prevents errors and ensures all notifications are heard

Fixes #20


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/RedWizardsLab/EchoHub/pull/36 **Author:** [@Stone-Red-Code](https://github.com/Stone-Red-Code) **Created:** 2/25/2026 **Status:** ✅ Merged **Merged:** 4/20/2026 **Merged by:** [@HueByte](https://github.com/HueByte) **Base:** `dev` ← **Head:** `dev_fix_notification_sound` --- ### 📝 Commits (2) - [`802db1c`](https://github.com/RedWizardsLab/EchoHub/commit/802db1ca09f5a0007164e269b935824cd2ac4c5c) fix: error when playing notification sounds in quick succession - [`ca97254`](https://github.com/RedWizardsLab/EchoHub/commit/ca97254a755b275c982ddebbea0dc4999a7c2a43) chore: add notification bug fix to changelog ### 📊 Changes **2 files changed** (+21 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `docs/changelog/v0.2.10.md` (+1 -0) 📝 `src/EchoHub.Client/Services/NotificationSoundService.cs` (+20 -3) </details> ### 📄 Description # Summary Fixes errors when playing notification sounds in quick succession. This fix may cause notification sounds to be delayed if the previous sound is still playing, but it prevents errors and ensures all notifications are heard Fixes #20 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Stone_Red added the pull-request label 2026-09-04 00:49:48 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RedWizardsLab/EchoHub#35