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

feat: add play sound config #856

Merged
merged 2 commits into from
Nov 15, 2024
Merged

feat: add play sound config #856

merged 2 commits into from
Nov 15, 2024

Conversation

ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Oct 28, 2024

☑️ Resolves

Desktop alternative for the following web settings (but with respect DND option):

image

For simple testing - check initial state

image

🖼️ Screenshots

image

@ShGKme ShGKme added enhancement New feature or request 2. developing labels Oct 28, 2024
@ShGKme ShGKme self-assigned this Oct 28, 2024
@nickvergessen
Copy link
Member

Ref nextcloud/spreed#13437 🙈

@ShGKme
Copy link
Contributor Author

ShGKme commented Oct 29, 2024

Ref nextcloud/spreed#13437 🙈

Wrong link? Or how is lobby related?

@nickvergessen
Copy link
Member

Check the description

@ShGKme ShGKme force-pushed the feat/config-play-sound branch from 4ac2d12 to 2785507 Compare October 30, 2024 14:08
@ShGKme ShGKme marked this pull request as ready for review October 30, 2024 14:09
Copy link

@DorraJaouad DorraJaouad left a comment

Choose a reason for hiding this comment

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

Tested

@ShGKme ShGKme mentioned this pull request Nov 8, 2024
@ShGKme
Copy link
Contributor Author

ShGKme commented Nov 8, 2024

Still fighting ESLint. I don't understand the error

@Antreesy Antreesy force-pushed the feat/config-play-sound branch from 2785507 to 11a70a6 Compare November 12, 2024 08:46
Copy link
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

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

Tested. init.js had unused imports after you moved user status listener

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme force-pushed the feat/config-play-sound branch from 11a70a6 to 85535bc Compare November 15, 2024 14:23
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Co-authored-by: Maksim Sukharev <antreesy.web@gmail.com>
@ShGKme ShGKme force-pushed the feat/config-play-sound branch from 85535bc to 4d15997 Compare November 15, 2024 14:26
@ShGKme
Copy link
Contributor Author

ShGKme commented Nov 15, 2024

Rebased onto main after a long time. No changes. @Antreesy's changes are included.

@ShGKme
Copy link
Contributor Author

ShGKme commented Nov 15, 2024

Still fighting ESLint. I don't understand the error

The error still exists but doesn't lead to fail eslint check...

@ShGKme
Copy link
Contributor Author

ShGKme commented Nov 15, 2024

Check the description

For clarification: that link is not related. This PR is only about call/notifications sound, same as available on the Web. Not about disabling any sounds completely.

@ShGKme ShGKme merged commit cd63aac into main Nov 15, 2024
10 checks passed
@ShGKme ShGKme deleted the feat/config-play-sound branch November 15, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot disable notification sound
4 participants