[17.0][ADD] mail_notification_sound_volume#162
Open
AndreuOForgeFlow wants to merge 1 commit intoOCA:17.0from
Open
[17.0][ADD] mail_notification_sound_volume#162AndreuOForgeFlow wants to merge 1 commit intoOCA:17.0from
AndreuOForgeFlow wants to merge 1 commit intoOCA:17.0from
Conversation
Contributor
GuillemCForgeFlow
left a comment
GuillemCForgeFlow
left a comment
There was a problem hiding this comment.
the idea seems really interesting 👀 but what about renaming to mail_notification_sound_volume to be more specific?
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_notification_volume
branch
from
323da02 to
200d7f8
Compare
OCA-git-bot
added
series:17.0
mod:mail_notification_sound_volume
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_notification_volume
branch
from
200d7f8 to
3d9964e
Compare
OCA-git-bot
added
mod:mail_notification_volume
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_notification_volume
branch
from
3d9964e to
5f9d723
Compare
OCA-git-bot
added
mod:mail_notification_sound_volume
AndreuOForgeFlow
changed the title
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_notification_volume
branch
from
5f9d723 to
80056f4
Compare
GuillemCForgeFlow
suggested changes
Apr 20, 2026
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_notification_volume
branch
2 times, most recently
from
b8417d9 to
e98b55f
Compare
GuillermoNforgeflow
force-pushed
the
17.0-add-mail_notification_volume
branch
from
e98b55f to
789f13d
Compare
GuillermoNforgeflow
approved these changes
May 7, 2026
GuillermoNforgeflow
left a comment
GuillermoNforgeflow
left a comment
There was a problem hiding this comment.
lgtm 👍
Code reviewed and tested
Author
GuillemCForgeFlow
approved these changes
May 7, 2026
Contributor
GuillemCForgeFlow
left a comment
GuillemCForgeFlow
left a comment
There was a problem hiding this comment.
lgtm code and functional review 👍🏿
Contributor
|
This PR has the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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%.