[19.0][ADD] subscription_oca_rating#1454
Open
alvaro-domatix wants to merge 1 commit into
Open
Conversation
OCA-git-bot
added
series:19.0
mod:subscription_oca_rating
alvaro-domatix
force-pushed
the
19.0-add-subscription_oca_rating
branch
from
5ec54b6 to
e039e8d
Compare
New bridge module between `subscription_oca` and the community `rating` addon. Inheriting `rating.mixin` on `sale.subscription` exposes the standard rating statistics (last value, average, count, satisfaction percentage, etc.) on the subscription and lets users see and search ratings via the standard Rating model. * `sale.subscription` extension adds `rating.mixin` (with the `_inherit` order `['rating.mixin', 'sale.subscription']` to keep the MRO consistent). * `action_send_rating_request` sends a configurable mail template to the subscription's partner. * Form view: header button to send the rating email + stat button linking to the standard rating list filtered for this contract. * Search view: "With ratings" filter. * Mail template `mail_template_subscription_rating_request` ships as loadable data, so it can be customised per instance. Lives in its own addon directory because `rating` is LGPL-3 and not acceptable in the core module dependencies (§0.3 of the project guidelines).
alvaro-domatix
force-pushed
the
19.0-add-subscription_oca_rating
branch
from
e039e8d to
9bab2ac
Compare
jorgeglez1990
approved these changes
Jun 24, 2026
Contributor
|
This PR has the |
Member
|
/ocabot merge nobump |
Contributor
|
On my way to merge this fine PR! |
Contributor
|
@yvaucher The merge process could not be finalized, because command |
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
5 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.
New bridge module between
subscription_ocaand theratingaddon.sale.subscriptioninheritsrating.mixin, so the standard rating statistics (last value, average, count, satisfaction percentage) become available on subscriptions. A header button sends a configurable rating request email to the customer, and the same request is sent automatically when the subscription is closed (both the close wizard and the cron go through it); a failure to send never blocks the close. The form gets a stat button to the ratings of the contract and the search view a "With ratings" filter.Shipped as a separate addon to keep the
ratingdependency out of the core module.