Skip to content

[17.0][ADD] mail_notification_sound_volume#162

Merged
OCA-git-bot merged 1 commit into
OCA:17.0from
ForgeFlow:17.0-add-mail_notification_volume
May 12, 2026
Merged

[17.0][ADD] mail_notification_sound_volume#162
OCA-git-bot merged 1 commit into
OCA:17.0from
ForgeFlow:17.0-add-mail_notification_volume

Conversation

@AndreuOForgeFlow
Copy link
Copy Markdown

@AndreuOForgeFlow AndreuOForgeFlow commented Mar 20, 2026 •
edited
Loading

This module allows users to configure the volume of notification sounds in Odoo (chat messages, inbox notifications, etc.).

A volume slider is added to the user's Preferences page, with a test button to preview the sound at the selected volume.

By default, the volume is set to 100%.

GuillemCForgeFlow
GuillemCForgeFlow reviewed Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@GuillemCForgeFlow GuillemCForgeFlow left a comment

Choose a reason for hiding this comment

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

the idea seems really interesting 👀 but what about renaming to mail_notification_sound_volume to be more specific?

@GuillermoNforgeflow GuillermoNforgeflow force-pushed the 17.0-add-mail_notification_volume branch from 323da02 to 200d7f8 Compare April 17, 2026 07:01
@OCA-git-bot OCA-git-bot added series:17.0 mod:mail_notification_sound_volume Module mail_notification_sound_volume labels Apr 17, 2026
@GuillermoNforgeflow GuillermoNforgeflow force-pushed the 17.0-add-mail_notification_volume branch from 200d7f8 to 3d9964e Compare April 17, 2026 07:45
@OCA-git-bot OCA-git-bot added mod:mail_notification_volume Module mail_notification_volume and removed mod:mail_notification_sound_volume Module mail_notification_sound_volume labels Apr 17, 2026
@GuillermoNforgeflow GuillermoNforgeflow force-pushed the 17.0-add-mail_notification_volume branch from 3d9964e to 5f9d723 Compare April 17, 2026 08:07
@OCA-git-bot OCA-git-bot added mod:mail_notification_sound_volume Module mail_notification_sound_volume and removed mod:mail_notification_volume Module mail_notification_volume labels Apr 17, 2026
@AndreuOForgeFlow AndreuOForgeFlow changed the title [17.0][ADD] mail_notification_volume [17.0][ADD] mail_notification_sound_volume Apr 17, 2026
@AndreuOForgeFlow AndreuOForgeFlow marked this pull request as ready for review April 17, 2026 08:29
@GuillermoNforgeflow GuillermoNforgeflow force-pushed the 17.0-add-mail_notification_volume branch from 5f9d723 to 80056f4 Compare April 17, 2026 09:41
GuillemCForgeFlow
GuillemCForgeFlow suggested changes Apr 20, 2026
View reviewed changes
Comment thread mail_notification_sound_volume/static/src/out_of_focus_service_patch.esm.js
Comment thread mail_notification_sound_volume/views/res_users_views.xml Outdated
@GuillermoNforgeflow GuillermoNforgeflow force-pushed the 17.0-add-mail_notification_volume branch 2 times, most recently from b8417d9 to e98b55f Compare April 20, 2026 07:23
@GuillermoNforgeflow GuillermoNforgeflow force-pushed the 17.0-add-mail_notification_volume branch from e98b55f to 789f13d Compare April 20, 2026 07:29
GuillermoNforgeflow
GuillermoNforgeflow approved these changes May 7, 2026
View reviewed changes
Copy link
Copy Markdown

@GuillermoNforgeflow GuillermoNforgeflow left a comment

Choose a reason for hiding this comment

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

lgtm 👍
Code reviewed and tested

@AndreuOForgeFlow
Copy link
Copy Markdown
Author

AndreuOForgeFlow commented May 7, 2026

Hi @etobella & @dreispt, could you review and possibly merge this one?

GuillemCForgeFlow
GuillemCForgeFlow approved these changes May 7, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@GuillemCForgeFlow GuillemCForgeFlow left a comment

Choose a reason for hiding this comment

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

lgtm code and functional review 👍🏿

@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented May 7, 2026

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@AndreuOForgeFlow
Copy link
Copy Markdown
Author

AndreuOForgeFlow commented May 12, 2026

Hi @pedrobaeza, could we possibly merge this one?

@pedrobaeza
Copy link
Copy Markdown
Member

pedrobaeza commented May 12, 2026

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented May 12, 2026

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-162-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 107d49d into OCA:17.0 May 12, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented May 12, 2026

Congratulations, your PR was merged at c668d1d. Thanks a lot for contributing to OCA. ❤️

@AndreuOForgeFlow AndreuOForgeFlow deleted the 17.0-add-mail_notification_volume branch May 12, 2026 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@GuillemCForgeFlow GuillemCForgeFlow GuillemCForgeFlow approved these changes

@GuillermoNforgeflow GuillermoNforgeflow GuillermoNforgeflow approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 mod:mail_notification_sound_volume Module mail_notification_sound_volume ready to merge series:17.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@AndreuOForgeFlow @OCA-git-bot @pedrobaeza @GuillemCForgeFlow @GuillermoNforgeflow