Skip to content

[19.0][MIG] mute_notification_user_autosubscribe: Migration to 19.0#1838

Open
pablo-cort-s73 wants to merge 4 commits intoOCA:19.0from
Studio73:19.0-mig-mute_notification_user_autosubscribe
Open

[19.0][MIG] mute_notification_user_autosubscribe: Migration to 19.0#1838
pablo-cort-s73 wants to merge 4 commits intoOCA:19.0from
Studio73:19.0-mig-mute_notification_user_autosubscribe

Conversation

@pablo-cort-s73
Copy link
Copy Markdown

PR Description - [MIG] mute_notification_user_autosubscribe: Migration to 19.0

Summary

Migration of the mute_notification_user_autosubscribe module to Odoo 19.0.

Technical Changes

res.users (Model)

  • Replaced user_has_groups() method with has_groups().
  • Updated search domains to use the new odoo.fields.Domain wrapper.

user.autosubscribe.mute (Model)

  • Refactored _get_user_models_domain method to construct and return an odoo.fields.Domain object instead of an evaluated f-string.
  • Updated model_id field domain definition to use a lambda function: domain=lambda self: self._get_user_models_domain.
  • Replaced the legacy _sql_constraints list with the new models.Constraint implementation for unique_model_id.

Views (user_autosubscribe_mute_views.xml)

  • Replaced the deprecated <tree> root node with <list> in the view architecture.

manuelregidor and others added 3 commits March 26, 2026 10:10
Currently translated at 100.0% (18 of 18 strings)

Translation: social-17.0/social-17.0-mute_notification_user_autosubscribe
Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mute_notification_user_autosubscribe/it/
@pablo-cort-s73 pablo-cort-s73 mentioned this pull request Mar 26, 2026
5 tasks
@pablo-cort-s73 pablo-cort-s73 force-pushed the 19.0-mig-mute_notification_user_autosubscribe branch 2 times, most recently from c10e7c3 to 32450f8 Compare March 26, 2026 10:20
@pablo-cort-s73 pablo-cort-s73 force-pushed the 19.0-mig-mute_notification_user_autosubscribe branch from 32450f8 to 8bfc96a Compare March 26, 2026 10:24
Copy link
Copy Markdown

@Reyes4711-S73 Reyes4711-S73 left a comment

Choose a reason for hiding this comment

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

LGTM
Functional review OK

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants