[FIX] document_page_approval: Add correct followers in history pages

[BOT] post-merge updates
This commit is contained in:
Víctor Martínez 2021-02-23 09:37:51 +01:00 committed by Bhavesh Heliconia
parent 72b41d914d
commit 279891c8f4
6 changed files with 49 additions and 14 deletions

View File

@ -7,7 +7,7 @@ Document Page Approval
!! This file is generated by oca-gen-addon-readme !!
!! changes will be overwritten. !!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!! source digest: sha256:ca1d6c5564a2a481b7722e98d0be8dd36b075b71592b62c54b782c6d584d3319
!! source digest: sha256:eaa85b29d79372636ce9dda9fbf87f567e6702f38a57385384a168b28a517ff5
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
@ -84,7 +84,11 @@ Contributors
* Gervais Naoussi <gervaisnaoussi@gmail.com>
* Maxime Chambreuil <mchambreuil@opensourceintegrators.com>
* Iván Todorovich <ivan.todorovich@gmail.com>
* Victor M.M. Torres <victor.martin@tecnativa.com>
* `Tecnativa <https://www.tecnativa.com>`_:
* Victor M.M. Torres
* Víctor Martínez
* `Guadaltech <https://www.guadaltech.es>`_:

View File

@ -3,7 +3,7 @@
{
"name": "Document Page Approval",
"version": "16.0.1.1.0",
"version": "16.0.1.1.1",
"author": "Savoir-faire Linux, Odoo Community Association (OCA)",
"website": "https://github.com/OCA/knowledge",
"license": "AGPL-3",

View File

@ -78,7 +78,7 @@ class DocumentPageHistory(models.Model):
users = self.env["res.users"].search(
[("groups_id", "in", guids), ("groups_id", "in", approver_gid.id)]
)
rec.message_subscribe([u.id for u in users])
rec.message_subscribe(partner_ids=users.mapped("partner_id").ids)
rec.message_post_with_template(template.id)
else:
# auto-approve if approval is not required

View File

@ -3,7 +3,11 @@
* Gervais Naoussi <gervaisnaoussi@gmail.com>
* Maxime Chambreuil <mchambreuil@opensourceintegrators.com>
* Iván Todorovich <ivan.todorovich@gmail.com>
* Victor M.M. Torres <victor.martin@tecnativa.com>
* `Tecnativa <https://www.tecnativa.com>`_:
* Victor M.M. Torres
* Víctor Martínez
* `Guadaltech <https://www.guadaltech.es>`_:

View File

@ -1,4 +1,3 @@
<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
@ -367,7 +366,7 @@ ul.auto-toc {
!! This file is generated by oca-gen-addon-readme !!
!! changes will be overwritten. !!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!! source digest: sha256:ca1d6c5564a2a481b7722e98d0be8dd36b075b71592b62c54b782c6d584d3319
!! source digest: sha256:eaa85b29d79372636ce9dda9fbf87f567e6702f38a57385384a168b28a517ff5
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/knowledge/tree/16.0/document_page_approval"><img alt="OCA/knowledge" src="https://img.shields.io/badge/github-OCA%2Fknowledge-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/knowledge-16-0/knowledge-16-0-document_page_approval"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/knowledge&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
<p>This module adds a workflow to approve page modifications
@ -426,14 +425,27 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
</div>
<div class="section" id="contributors">
<h2><a class="toc-backref" href="#toc-entry-6">Contributors</a></h2>
<ul>
<li><p class="first">Odoo SA &lt;<a class="reference external" href="mailto:info&#64;odoo.com">info&#64;odoo.com</a>&gt;</p>
</li>
<li><p class="first">Savoir-faire Linux &lt;<a class="reference external" href="mailto:support&#64;savoirfairelinux.com">support&#64;savoirfairelinux.com</a>&gt;</p>
</li>
<li><p class="first">Gervais Naoussi &lt;<a class="reference external" href="mailto:gervaisnaoussi&#64;gmail.com">gervaisnaoussi&#64;gmail.com</a>&gt;</p>
</li>
<li><p class="first">Maxime Chambreuil &lt;<a class="reference external" href="mailto:mchambreuil&#64;opensourceintegrators.com">mchambreuil&#64;opensourceintegrators.com</a>&gt;</p>
</li>
<li><p class="first">Iván Todorovich &lt;<a class="reference external" href="mailto:ivan.todorovich&#64;gmail.com">ivan.todorovich&#64;gmail.com</a>&gt;</p>
</li>
<li><p class="first"><a class="reference external" href="https://www.tecnativa.com">Tecnativa</a>:</p>
<blockquote>
<ul class="simple">
<li>Victor M.M. Torres</li>
<li>Víctor Martínez</li>
</ul>
</blockquote>
</li>
<li><p class="first"><a class="reference external" href="https://www.guadaltech.es">Guadaltech</a>:</p>
<ul class="simple">
<li>Odoo SA &lt;<a class="reference external" href="mailto:info&#64;odoo.com">info&#64;odoo.com</a>&gt;</li>
<li>Savoir-faire Linux &lt;<a class="reference external" href="mailto:support&#64;savoirfairelinux.com">support&#64;savoirfairelinux.com</a>&gt;</li>
<li>Gervais Naoussi &lt;<a class="reference external" href="mailto:gervaisnaoussi&#64;gmail.com">gervaisnaoussi&#64;gmail.com</a>&gt;</li>
<li>Maxime Chambreuil &lt;<a class="reference external" href="mailto:mchambreuil&#64;opensourceintegrators.com">mchambreuil&#64;opensourceintegrators.com</a>&gt;</li>
<li>Iván Todorovich &lt;<a class="reference external" href="mailto:ivan.todorovich&#64;gmail.com">ivan.todorovich&#64;gmail.com</a>&gt;</li>
<li>Victor M.M. Torres &lt;<a class="reference external" href="mailto:victor.martin&#64;tecnativa.com">victor.martin&#64;tecnativa.com</a>&gt;</li>
<li><a class="reference external" href="https://www.guadaltech.es">Guadaltech</a>:<ul>
<li>Fernando La Chica &lt;<a class="reference external" href="mailto:fernando.lachica&#64;guadaltech.es">fernando.lachica&#64;guadaltech.es</a>&gt;</li>
</ul>
</li>

View File

@ -13,6 +13,18 @@ class TestDocumentPageApproval(common.TransactionCase):
"document_page_approval.group_document_approver_user"
)
self.env.ref("base.user_root").write({"groups_id": [(4, self.approver_gid.id)]})
self.env.ref("base.user_admin").write(
{"groups_id": [(4, self.approver_gid.id)]}
)
self.user2 = self.env["res.users"].create(
{
"name": "Test user 2",
"login": "Test user 2",
"groups_id": [
(6, 0, [self.env.ref("base.group_user").id, self.approver_gid.id])
],
}
)
# demo_approval
self.category2 = self.page_obj.create(
{
@ -44,6 +56,9 @@ class TestDocumentPageApproval(common.TransactionCase):
# It should automatically be in 'to approve' state
self.assertEqual(chreq.state, "to approve")
user_admin = self.env.ref("base.user_admin")
self.assertTrue(user_admin.partner_id.id in chreq.message_partner_ids.ids)
self.assertTrue(self.user2.partner_id.id in chreq.message_partner_ids.ids)
# Needed to compute calculated fields
page.invalidate_model()