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

[feature](web-app): Add Alarm Voice Alerts #2906

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

[feature](web-app): Add Alarm Voice Alerts #2906

wants to merge 8 commits into from

Conversation

zqr10159
Copy link
Member

What's changed?

Add alarm voice prompts, off by default
image

image

Checklist

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

- Implement AlertSoundService to handle audio playback
- Update NotifyComponent to use AlertSoundService for playing alert sounds
- Add test button for alert sound in NotifyComponent
- Configure audio asset in angular.json
- Add mute functionality for alert sound
- Update alert sound for different languages
- Improve error handling when playing sound
- Add new icon for mute status
- Update notification component to include mute toggle
- Refresh notifications every 10 seconds instead of30
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

license-eye has checked 2898 files.

Valid Invalid Ignored Fixed
1500 1 1397 0
Click to see the invalid file list
  • web-app/src/app/service/alert-sound.service.ts
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

zqr10159 and others added 2 commits December 26, 2024 23:49
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Logic <[email protected]>
@zqr10159 zqr10159 added this to the 1.7.0 milestone Dec 27, 2024
@zqr10159 zqr10159 marked this pull request as draft December 27, 2024 00:42
zqr10159 and others added 2 commits December 27, 2024 15:41
… represent mute configuration

- Implement MuteGeneralConfigServiceImpl to handle mute config
- Update ConfigInitializer to initialize mute config- Add mute config support in web-app
- Update notify component to use mute config for sound control
@zqr10159 zqr10159 marked this pull request as ready for review December 27, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants