From 47a40a442ccaddb9edda737dbf89b6a781c80fc5 Mon Sep 17 00:00:00 2001
From: Antoni Marroig Campomar <amarroig@apsl.net>
Date: Fri, 17 May 2024 17:43:33 +0200
Subject: [PATCH] [MIG] sign_oca: Migration to 17.0 > > Co-authored-by: Bernat
 Puig <bernat.puig@forgeflow.com>

---
 sign_oca/__manifest__.py                      |  4 +-
 sign_oca/controllers/main.py                  |  3 +-
 .../migrations/16.0.1.1.1/pre-migration.py    | 15 ----
 sign_oca/models/__init__.py                   |  1 +
 sign_oca/models/res_company.py                | 14 +++
 sign_oca/models/sign_oca_request.py           | 86 ++++++++++++-------
 .../sign_oca_configure.esm.js                 | 39 +++++----
 .../sign_oca_configure/sign_oca_configure.xml |  1 -
 .../sign_oca_pdf/sign_oca_pdf.esm.js          |  9 +-
 .../sign_oca_pdf/sign_oca_pdf_action.esm.js   | 27 +++---
 .../sign_oca_pdf_common.esm.js                | 17 ++--
 .../sign_oca_pdf_common.xml                   |  2 +-
 .../sign_oca_pdf_common_action.esm.js         | 25 +++---
 .../sign_oca_pdf_portal.esm.js                | 35 +++++---
 .../sign_oca_pdf_portal.xml                   |  2 +-
 sign_oca/static/src/elements/check.esm.js     |  9 +-
 sign_oca/static/src/elements/signature.esm.js |  6 +-
 sign_oca/static/src/elements/text.esm.js      |  7 +-
 sign_oca/static/src/js/main.esm.js            |  7 --
 sign_oca/static/src/js/sign_oca.esm.js        | 18 ++--
 .../src/js/signer_menu_container.esm.js       | 10 +--
 .../static/src/js/signer_menu_view.esm.js     | 12 +--
 sign_oca/static/src/js/systray.esm.js         |  3 +-
 sign_oca/static/src/js/systray_service.esm.js |  1 +
 .../static/src/xml/signer_menu_container.xml  |  2 +-
 sign_oca/static/src/xml/systray.xml           |  2 +-
 sign_oca/templates/assets.xml                 |  1 -
 sign_oca/tests/test_sign.py                   | 15 +++-
 sign_oca/tests/test_sign_portal.py            | 31 ++++---
 sign_oca/views/res_partner_views.xml          |  2 +-
 sign_oca/views/sign_oca_request.xml           | 42 +++++----
 sign_oca/views/sign_oca_request_log.xml       |  8 +-
 sign_oca/views/sign_oca_role.xml              |  6 +-
 sign_oca/wizards/__init__.py                  |  1 +
 sign_oca/wizards/res_config_settings.py       | 12 +++
 .../wizards/res_config_settings_views.xml     | 29 +++++++
 .../wizards/sign_oca_template_generate.xml    |  7 +-
 37 files changed, 299 insertions(+), 212 deletions(-)
 delete mode 100644 sign_oca/migrations/16.0.1.1.1/pre-migration.py
 create mode 100644 sign_oca/models/res_company.py
 delete mode 100644 sign_oca/static/src/js/main.esm.js
 create mode 100644 sign_oca/wizards/res_config_settings.py
 create mode 100644 sign_oca/wizards/res_config_settings_views.xml

diff --git a/sign_oca/__manifest__.py b/sign_oca/__manifest__.py
index 0e395fb7..b515c739 100644
--- a/sign_oca/__manifest__.py
+++ b/sign_oca/__manifest__.py
@@ -5,7 +5,7 @@
     "name": "Sign Oca",
     "summary": """
         Allow to sign documents inside Odoo CE""",
-    "version": "16.0.2.0.0",
+    "version": "17.0.1.0.0",
     "license": "AGPL-3",
     "author": "Dixmit,Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/sign",
@@ -14,6 +14,7 @@
         "security/security.xml",
         "views/menu.xml",
         "data/data.xml",
+        "wizards/res_config_settings_views.xml",
         "wizards/sign_oca_template_generate.xml",
         "wizards/sign_oca_template_generate_multi.xml",
         "views/res_partner_views.xml",
@@ -61,7 +62,6 @@
             "sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.esm.js",
             "sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.xml",
             "sign_oca/static/src/scss/portal.scss",
-            "sign_oca/static/src/js/*.js",
             "sign_oca/static/src/xml/*.xml",
         ],
         "sign_oca.sign_assets": [
diff --git a/sign_oca/controllers/main.py b/sign_oca/controllers/main.py
index 920940f5..8e471579 100644
--- a/sign_oca/controllers/main.py
+++ b/sign_oca/controllers/main.py
@@ -101,5 +101,4 @@ def get_sign_oca_sign_access(self, signer_id, access_token, items):
             )
         except (AccessError, MissingError):
             return request.redirect("/my")
-        signer_sudo.action_sign(items, access_token=access_token)
-        return True
+        return signer_sudo.action_sign(items, access_token=access_token)
diff --git a/sign_oca/migrations/16.0.1.1.1/pre-migration.py b/sign_oca/migrations/16.0.1.1.1/pre-migration.py
deleted file mode 100644
index 9daf67cd..00000000
--- a/sign_oca/migrations/16.0.1.1.1/pre-migration.py
+++ /dev/null
@@ -1,15 +0,0 @@
-# Copyright 2024 ForgeFlow <http://www.forgeflow.com>
-# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
-
-import logging
-
-_logger = logging.getLogger(__name__)
-
-
-def migrate(cr, version):
-    _logger.info("Change partner_type field to partner_selection_policy.")
-    query = (
-        "ALTER TABLE sign_oca_role RENAME COLUMN"
-        " 'partner_type' TO 'partner_selection_policy'"
-    )
-    cr.execute(query)
diff --git a/sign_oca/models/__init__.py b/sign_oca/models/__init__.py
index 32aa7235..1cbaaba0 100644
--- a/sign_oca/models/__init__.py
+++ b/sign_oca/models/__init__.py
@@ -1,3 +1,4 @@
+from . import res_company
 from . import res_users
 from . import res_partner
 from . import sign_oca_template
diff --git a/sign_oca/models/res_company.py b/sign_oca/models/res_company.py
new file mode 100644
index 00000000..6b21ca54
--- /dev/null
+++ b/sign_oca/models/res_company.py
@@ -0,0 +1,14 @@
+# Copyright 2024 ForgeFlow S.L. (http://www.forgeflow.com)
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html).
+
+from odoo import fields, models
+
+
+class ResCompany(models.Model):
+    _inherit = "res.company"
+
+    sign_oca_send_sign_request_copy = fields.Boolean(
+        string="Send signers a copy of the final signed document",
+        help="Once all signers have signed the request, a copy of "
+        "the final document will be sent to each of them.",
+    )
diff --git a/sign_oca/models/sign_oca_request.py b/sign_oca/models/sign_oca_request.py
index b6300e6b..a601aa0f 100644
--- a/sign_oca/models/sign_oca_request.py
+++ b/sign_oca/models/sign_oca_request.py
@@ -24,16 +24,12 @@ class SignOcaRequest(models.Model):
 
     name = fields.Char(required=True)
     active = fields.Boolean(default=True)
-    template_id = fields.Many2one("sign.oca.template", readonly=True)
-    data = fields.Binary(
-        required=True, readonly=True, states={"draft": [("readonly", False)]}
-    )
+    template_id = fields.Many2one("sign.oca.template")
+    data = fields.Binary(required=True)
     filename = fields.Char()
     user_id = fields.Many2one(
         comodel_name="res.users",
         string="Responsible",
-        readonly=True,
-        states={"draft": [("readonly", False)]},
         default=lambda self: self.env.user,
         required=True,
     )
@@ -45,8 +41,6 @@ class SignOcaRequest(models.Model):
             )
         ],
         string="Object",
-        readonly=True,
-        states={"draft": [("readonly", False)]},
     )
     signed = fields.Boolean(copy=False)
     signer_ids = fields.One2many(
@@ -54,8 +48,6 @@ class SignOcaRequest(models.Model):
         inverse_name="request_id",
         auto_join=True,
         copy=True,
-        readonly=True,
-        states={"draft": [("readonly", False)]},
     )
     signer_id = fields.Many2one(
         comodel_name="sign.oca.request.signer",
@@ -70,7 +62,6 @@ class SignOcaRequest(models.Model):
             ("cancel", "Cancelled"),
         ],
         default="draft",
-        readonly=True,
         required=True,
         copy=False,
         tracking=True,
@@ -80,16 +71,13 @@ class SignOcaRequest(models.Model):
     to_sign = fields.Boolean(compute="_compute_to_sign")
     signatory_data = fields.Serialized(
         default=lambda r: {},
-        readonly=True,
         copy=False,
     )
-    current_hash = fields.Char(readonly=True, copy=False)
+    current_hash = fields.Char(copy=False)
     company_id = fields.Many2one(
         "res.company",
         default=lambda r: r.env.company.id,
         required=True,
-        readonly=True,
-        states={"draft": [("readonly", False)]},
     )
     next_item_id = fields.Integer(compute="_compute_next_item_id")
 
@@ -264,6 +252,40 @@ def action_send(self, sign_now=False, message=""):
                 email_layout_xmlid="mail.mail_notification_light",
             )
 
+    def action_send_signed_request(self):
+        self.ensure_one()
+        if (
+            self.state != "signed"
+            or not self.env.company.sign_oca_send_sign_request_copy
+        ):
+            return
+        for signer in self.signer_ids:
+            attachments = (
+                self.env["ir.attachment"]
+                .sudo()
+                .search(
+                    [
+                        ("res_model", "=", "sign.oca.request"),
+                        ("res_id", "=", self.id),
+                        ("res_field", "=", "data"),
+                    ]
+                )
+            )
+            # The message will not be linked to the record because we do not want
+            # it happen.
+            self.env["mail.thread"].message_notify(
+                body=_(
+                    "%(name)s (%(email)s) has sent the signed document.",
+                    name=self.create_uid.name,
+                    email=self.create_uid.email,
+                ),
+                partner_ids=signer.partner_id.ids,
+                subject=_("Signed document"),
+                subtype_id=self.env.ref("mail.mt_comment").id,
+                mail_auto_delete=False,
+                attachment_ids=attachments.ids,
+            )
+
     def _check_signed(self):
         self.ensure_one()
         if self.state != "sent":
@@ -310,8 +332,8 @@ class SignOcaRequestSigner(models.Model):
     partner_name = fields.Char(related="partner_id.name")
     partner_id = fields.Many2one("res.partner", required=True, ondelete="restrict")
     role_id = fields.Many2one("sign.oca.role", required=True, ondelete="restrict")
-    signed_on = fields.Datetime(readonly=True)
-    signature_hash = fields.Char(readonly=True)
+    signed_on = fields.Datetime()
+    signature_hash = fields.Char()
     model = fields.Char(compute="_compute_model", store=True)
     res_id = fields.Integer(compute="_compute_res_id", store=True)
     is_allow_signature = fields.Boolean(compute="_compute_is_allow_signature")
@@ -358,10 +380,10 @@ def get_info(self, access_token=False):
             "items": self.request_id.signatory_data,
             "to_sign": self.request_id.to_sign,
             "partner": {
-                "id": self.env.user.partner_id.id,
-                "name": self.env.user.partner_id.name,
-                "email": self.env.user.partner_id.email,
-                "phone": self.env.user.partner_id.phone,
+                "id": self.partner_id.id,
+                "name": self.partner_id.name,
+                "email": self.partner_id.email,
+                "phone": self.partner_id.phone,
             },
         }
 
@@ -422,7 +444,11 @@ def action_sign(self, items, access_token=False):
         self.signature_hash = final_hash
         self.request_id._check_signed()
         self._set_action_log("sign", access_token=access_token)
-        # TODO: Add a return
+        self.request_id.action_send_signed_request()
+        return {
+            "type": "ir.actions.act_url",
+            "url": self.access_url,
+        }
 
     def _check_signable(self, item):
         if not item["required"]:
@@ -510,9 +536,9 @@ def _set_action_log(self, action, **kwargs):
         self.ensure_one()
         return self.request_id._set_action_log(action, signer_id=self.id, **kwargs)
 
-    def name_get(self):
-        result = [(signer.id, (signer.partner_id.display_name)) for signer in self]
-        return result
+    def _compute_display_name(self):
+        for signer in self:
+            signer.display_name = signer.partner_id.display_name
 
 
 class SignRequestLog(models.Model):
@@ -524,13 +550,10 @@ class SignRequestLog(models.Model):
     uid = fields.Many2one(
         "res.users",
         required=True,
-        readonly=True,
         ondelete="cascade",
         default=lambda r: r.env.user.id,
     )
-    date = fields.Datetime(
-        required=True, readonly=True, default=lambda r: fields.Datetime.now()
-    )
+    date = fields.Datetime(required=True, default=lambda r: fields.Datetime.now())
     partner_id = fields.Many2one(
         "res.partner", required=True, default=lambda r: r.env.user.partner_id.id
     )
@@ -549,7 +572,6 @@ class SignRequestLog(models.Model):
             ("configure", "Configure"),
         ],
         required=True,
-        readonly=True,
     )
-    access_token = fields.Char(readonly=True)
-    ip = fields.Char(readonly=True)
+    access_token = fields.Char()
+    ip = fields.Char()
diff --git a/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.esm.js b/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.esm.js
index 5178eb92..ace400de 100644
--- a/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.esm.js
+++ b/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.esm.js
@@ -1,12 +1,14 @@
-/** @odoo-module **/
+/** @odoo-module QWeb **/
 
-import {ComponentWrapper} from "web.OwlCompatibility";
-import AbstractAction from "web.AbstractAction";
-import Dialog from "web.Dialog";
-import core from "web.core";
-import ControlPanel from "web.ControlPanel";
+import {Component} from "@odoo/owl";
+import {ControlPanel} from "@web/search/control_panel/control_panel";
+import {Dialog} from "@web/core/dialog/dialog";
+import {FormRenderer} from "@web/views/form/form_renderer";
 import SignOcaPdfCommon from "../sign_oca_pdf_common/sign_oca_pdf_common.esm.js";
-const _t = core._t;
+import {_t} from "@web/core/l10n/translation";
+import {registry} from "@web/core/registry";
+import {renderToString} from "@web/core/utils/render";
+
 export class SignOcaConfigureControlPanel extends ControlPanel {}
 SignOcaConfigureControlPanel.template = "sign_oca.SignOcaConfigureControlPanel";
 export class SignOcaConfigure extends SignOcaPdfCommon {
@@ -35,7 +37,7 @@ export class SignOcaConfigure extends SignOcaPdfCommon {
                     }
                     var position = page.getBoundingClientRect();
                     this.contextMenu = $(
-                        core.qweb.render("sign_oca.sign_iframe_contextmenu", {
+                        renderToString("sign_oca.sign_iframe_contextmenu", {
                             page,
                             e,
                             left: ((e.pageX - position.x) * 100) / position.width + "%",
@@ -111,7 +113,7 @@ export class SignOcaConfigure extends SignOcaPdfCommon {
                 var dialog = new Dialog(this, {
                     title: _t("Edit field"),
                     $content: $(
-                        core.qweb.render("sign_oca.sign_oca_field_edition", {
+                        renderToString("sign_oca.sign_oca_field_edition", {
                             item,
                             info: this.info,
                         })
@@ -367,9 +369,8 @@ export class SignOcaConfigure extends SignOcaPdfCommon {
     }
 }
 
-export const SignOcaConfigureAction = AbstractAction.extend({
-    hasControlPanel: true,
-    init: function (parent, action) {
+export class SignOcaConfigureAction extends Component {
+    init(parent, action) {
         this._super.apply(this, arguments);
         this.model =
             (action.params.res_model !== undefined && action.params.res_model) ||
@@ -377,24 +378,24 @@ export const SignOcaConfigureAction = AbstractAction.extend({
         this.res_id =
             (action.params.res_id !== undefined && action.params.res_id) ||
             action.context.params.id;
-    },
+    }
     async start() {
         await this._super(...arguments);
-        this.component = new ComponentWrapper(this, SignOcaConfigure, {
+        this.component = new FormRenderer(this, SignOcaConfigure, {
             model: this.model,
             res_id: this.res_id,
         });
         this.$el.addClass("o_sign_oca_action");
         return this.component.mount(this.$(".o_content")[0]);
-    },
-    getState: function () {
+    }
+    getState() {
         var result = this._super(...arguments);
         result = _.extend({}, result, {
             res_model: this.model,
             res_id: this.res_id,
         });
         return result;
-    },
-});
-core.action_registry.add("sign_oca_configure", SignOcaConfigureAction);
+    }
+}
 SignOcaConfigure.template = "sign_oca.SignOcaConfigure";
+registry.category("actions").add("sign_oca_configure", SignOcaConfigureAction);
diff --git a/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.xml b/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.xml
index 01801d67..15bf4b1c 100644
--- a/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.xml
+++ b/sign_oca/static/src/components/sign_oca_configure/sign_oca_configure.xml
@@ -107,7 +107,6 @@
         t-name="sign_oca.SignOcaConfigure"
         t-inherit="sign_oca.SignOcaPdfCommon"
         t-inherit-mode="primary"
-        owl="1"
     >
         <xpath expr="//iframe" position="before">
             <div
diff --git a/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf.esm.js b/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf.esm.js
index d44bd89a..26184782 100644
--- a/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf.esm.js
+++ b/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf.esm.js
@@ -1,9 +1,10 @@
-/** @odoo-module **/
+/** @odoo-module Qweb **/
 
 import SignOcaPdfCommon from "../sign_oca_pdf_common/sign_oca_pdf_common.esm.js";
-import core from "web.core";
 import {registry} from "@web/core/registry";
 const SignRegistry = registry.category("sign_oca");
+import {renderToString} from "@web/core/utils/render";
+
 export default class SignOcaPdf extends SignOcaPdfCommon {
     setup() {
         super.setup(...arguments);
@@ -23,7 +24,7 @@ export default class SignOcaPdf extends SignOcaPdfCommon {
     }
     renderButtons(to_sign) {
         var $buttons = $(
-            core.qweb.render("oca_sign_oca.SignatureButtons", {
+            renderToString("oca_sign_oca.SignatureButtons", {
                 to_sign: to_sign,
             })
         );
@@ -86,7 +87,7 @@ export default class SignOcaPdf extends SignOcaPdfCommon {
             _.filter(this.info.items, (item) => {
                 return (
                     item.required &&
-                    item.role === this.info.role &&
+                    item.role_id === this.info.role_id &&
                     !SignRegistry.get(item.field_type).check(item)
                 );
             }).length === 0;
diff --git a/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf_action.esm.js b/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf_action.esm.js
index eac7195d..8dc83b1c 100644
--- a/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf_action.esm.js
+++ b/sign_oca/static/src/components/sign_oca_pdf/sign_oca_pdf_action.esm.js
@@ -1,14 +1,12 @@
 /** @odoo-module **/
 
-import AbstractAction from "web.AbstractAction";
-import {ComponentWrapper} from "web.OwlCompatibility";
+import {Component} from "@odoo/owl";
+import {FormRenderer} from "@web/views/form/form_renderer";
 import SignOcaPdf from "./sign_oca_pdf.esm.js";
-import core from "web.core";
+import {registry} from "@web/core/registry";
 
-const SignOcaPdfAction = AbstractAction.extend({
-    className: "o_sign_oca_content",
-    hasControlPanel: true,
-    init: function (parent, action) {
+export class SignOcaPdfAction extends Component {
+    init(parent, action) {
         this._super.apply(this, arguments);
         this.model =
             (action.params.res_model !== undefined && action.params.res_model) ||
@@ -16,25 +14,24 @@ const SignOcaPdfAction = AbstractAction.extend({
         this.res_id =
             (action.params.res_id !== undefined && action.params.res_id) ||
             action.context.params.id;
-    },
+    }
     async start() {
         await this._super(...arguments);
-        this.component = new ComponentWrapper(this, SignOcaPdf, {
+        this.component = new FormRenderer(this, SignOcaPdf, {
             model: this.model,
             res_id: this.res_id,
             updateControlPanel: this.updateControlPanel.bind(this),
             trigger: this.trigger_up.bind(this),
         });
         return this.component.mount(this.$(".o_content")[0]);
-    },
-    getState: function () {
+    }
+    getState() {
         var result = this._super(...arguments);
         result = _.extend({}, result, {
             res_model: this.model,
             res_id: this.res_id,
         });
         return result;
-    },
-});
-core.action_registry.add("sign_oca", SignOcaPdfAction);
-export default SignOcaPdfAction;
+    }
+}
+registry.category("actions").add("sign_oca", SignOcaPdfAction);
diff --git a/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.esm.js b/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.esm.js
index b0099e71..1adf6bc1 100644
--- a/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.esm.js
+++ b/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.esm.js
@@ -1,8 +1,9 @@
-/** @odoo-module **/
-const {Component, onMounted, onWillStart, onWillUnmount, useRef} = owl;
-import Dialog from "web.Dialog";
-import core from "web.core";
-const _t = core._t;
+/** @odoo-module QWeb **/
+import {Component, onMounted, onWillStart, onWillUnmount, useRef} from "@odoo/owl";
+import {Dialog} from "@web/core/dialog/dialog";
+import {_t} from "@web/core/l10n/translation";
+import {renderToString} from "@web/core/utils/render";
+
 export default class SignOcaPdfCommon extends Component {
     setup() {
         super.setup(...arguments);
@@ -11,8 +12,8 @@ export default class SignOcaPdfCommon extends Component {
         this.pdf_url = this.getPdfUrl();
         this.viewer_url = "/web/static/lib/pdfjs/web/viewer.html?file=" + this.pdf_url;
         this.iframe = useRef("sign_oca_iframe");
-        var iframeResolve = undefined;
-        var iframeReject = undefined;
+        var iframeResolve = "";
+        var iframeReject = "";
         this.iframeLoaded = new Promise(function (resolve, reject) {
             iframeResolve = resolve;
             iframeReject = reject;
@@ -112,7 +113,7 @@ export default class SignOcaPdfCommon extends Component {
                 item.page - 1
             ];
         var signatureItem = $(
-            core.qweb.render(this.field_template, {
+            renderToString(this.field_template, {
                 ...item,
             })
         );
diff --git a/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.xml b/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.xml
index 2a313a14..5a9b0638 100644
--- a/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.xml
+++ b/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <templates xml:space="preserve">
-    <t t-name="sign_oca.SignOcaPdfCommon" owl="1">
+    <t t-name="sign_oca.SignOcaPdfCommon">
         <div class="o_sign_oca_content">
             <iframe
                 class="o_sign_oca_iframe"
diff --git a/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common_action.esm.js b/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common_action.esm.js
index 99de1302..91166329 100644
--- a/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common_action.esm.js
+++ b/sign_oca/static/src/components/sign_oca_pdf_common/sign_oca_pdf_common_action.esm.js
@@ -1,13 +1,12 @@
 /** @odoo-module **/
 
-import AbstractAction from "web.AbstractAction";
-import {ComponentWrapper} from "web.OwlCompatibility";
+import {Component} from "@odoo/owl";
+import {FormRenderer} from "@web/views/form/form_renderer";
 import SignOcaPdfCommon from "./sign_oca_pdf_common.esm.js";
-import core from "web.core";
+import {registry} from "@web/core/registry";
 
-export const SignOcaPdfCommonAction = AbstractAction.extend({
-    hasControlPanel: true,
-    init: function (parent, action) {
+export class SignOcaPdfCommonAction extends Component {
+    init(parent, action) {
         this._super.apply(this, arguments);
         this.model =
             (action.params.res_model !== undefined && action.params.res_model) ||
@@ -15,23 +14,23 @@ export const SignOcaPdfCommonAction = AbstractAction.extend({
         this.res_id =
             (action.params.res_id !== undefined && action.params.res_id) ||
             action.context.params.id;
-    },
+    }
     async start() {
         await this._super(...arguments);
-        this.component = new ComponentWrapper(this, SignOcaPdfCommon, {
+        this.component = new FormRenderer(this, SignOcaPdfCommon, {
             model: this.model,
             res_id: this.res_id,
         });
         this.$el.addClass("o_sign_oca_action");
         return this.component.mount(this.$(".o_content")[0]);
-    },
-    getState: function () {
+    }
+    getState() {
         var result = this._super(...arguments);
         result = _.extend({}, result, {
             res_model: this.model,
             res_id: this.res_id,
         });
         return result;
-    },
-});
-core.action_registry.add("sign_oca_preview", SignOcaPdfCommonAction);
+    }
+}
+registry.category("actions").add("sign_oca_preview", SignOcaPdfCommonAction);
diff --git a/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.esm.js b/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.esm.js
index 29b4d546..186a57da 100644
--- a/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.esm.js
+++ b/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.esm.js
@@ -1,13 +1,12 @@
 /** @odoo-module **/
 
 const {App, mount, useRef} = owl;
-
 import SignOcaPdf from "../sign_oca_pdf/sign_oca_pdf.esm.js";
-
-import env from "web.public_env";
+import {makeEnv} from "@web/env";
 import {renderToString} from "@web/core/utils/render";
-import session from "web.session";
+import {session} from "@web/session";
 import {templates} from "@web/core/assets";
+
 export class SignOcaPdfPortal extends SignOcaPdf {
     setup() {
         super.setup(...arguments);
@@ -41,14 +40,25 @@ export class SignOcaPdfPortal extends SignOcaPdf {
         this.checkFilledAll();
     }
     _onClickSign() {
-        this.env.services.rpc({
-            route:
-                "/sign_oca/sign/" +
-                this.props.signer_id +
-                "/" +
-                this.props.access_token,
-            params: {items: this.info.items},
-        });
+        this.env.services
+            .rpc({
+                route:
+                    "/sign_oca/sign/" +
+                    this.props.signer_id +
+                    "/" +
+                    this.props.access_token,
+                params: {items: this.info.items},
+            })
+            .then((action) => {
+                // As we are on frontend env, it is not possible to use do_action(), so we
+                // redirect to the corresponding URL or reload the page if the action is not
+                // an url.
+                if (action.type === "ir.actions.act_url") {
+                    window.location = action.url;
+                } else {
+                    window.location.reload();
+                }
+            });
     }
 }
 SignOcaPdfPortal.template = "sign_oca.SignOcaPdfPortal";
@@ -63,6 +73,7 @@ export function initDocumentToSign(properties) {
         ]).then(async function () {
             var app = new App(null, {templates, test: true});
             renderToString.app = app;
+            const env = makeEnv();
             mount(SignOcaPdfPortal, document.body, {
                 env,
                 props: properties,
diff --git a/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.xml b/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.xml
index 3d2c8169..beec942f 100644
--- a/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.xml
+++ b/sign_oca/static/src/components/sign_oca_pdf_portal/sign_oca_pdf_portal.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <templates xml:space="preserve">
-    <div t-name="sign_oca.SignOcaPdfPortal" owl="1" class="o_sign_oca_content">
+    <div t-name="sign_oca.SignOcaPdfPortal" class="o_sign_oca_content">
         <header>
             <div class="container-fluid">
                 <div class="d-flex justify-content-between flex-fill">
diff --git a/sign_oca/static/src/elements/check.esm.js b/sign_oca/static/src/elements/check.esm.js
index 9a6a5a2b..bb17b8a6 100644
--- a/sign_oca/static/src/elements/check.esm.js
+++ b/sign_oca/static/src/elements/check.esm.js
@@ -1,7 +1,8 @@
-/** @odoo-module **/
+/** @odoo-module Qweb **/
 
-import core from "web.core";
 import {registry} from "@web/core/registry";
+import {renderToString} from "@web/core/utils/render";
+
 const checkSignOca = {
     change: function (value, parent, item) {
         item.value = value;
@@ -9,7 +10,7 @@ const checkSignOca = {
     },
     generate: function (parent, item, signatureItem) {
         var input = $(
-            core.qweb.render("sign_oca.sign_iframe_field_check", {
+            renderToString("sign_oca.sign_iframe_field_check", {
                 item: item,
                 role_id: parent.info.role_id,
             })
@@ -42,7 +43,7 @@ const checkSignOca = {
             ev.preventDefault();
             var next_items = _.filter(
                 parent.info.items,
-                (i) => i.tabindex > item.tabindex && i.role_id === parent.role_id
+                (i) => i.tabindex > item.tabindex && i.role_id === parent.info.role_id
             ).sort((a, b) => a.tabindex - b.tabindex);
             if (next_items.length > 0) {
                 ev.currentTarget.blur();
diff --git a/sign_oca/static/src/elements/signature.esm.js b/sign_oca/static/src/elements/signature.esm.js
index 2a805205..7e0777eb 100644
--- a/sign_oca/static/src/elements/signature.esm.js
+++ b/sign_oca/static/src/elements/signature.esm.js
@@ -1,8 +1,8 @@
-/** @odoo-module **/
+/** @odoo-module Qweb **/
 
 import {SignatureDialog} from "@web/core/signature/signature_dialog";
-import core from "web.core";
 import {registry} from "@web/core/registry";
+import {renderToString} from "@web/core/utils/render";
 
 const signatureSignOca = {
     uploadSignature: function (parent, item, signatureItem, data) {
@@ -19,7 +19,7 @@ const signatureSignOca = {
     },
     generate: function (parent, item, signatureItem) {
         var input = $(
-            core.qweb.render("sign_oca.sign_iframe_field_signature", {item: item})
+            renderToString("sign_oca.sign_iframe_field_signature", {item: item})
         )[0];
         if (item.role_id === parent.info.role_id) {
             signatureItem[0].addEventListener("focus_signature", () => {
diff --git a/sign_oca/static/src/elements/text.esm.js b/sign_oca/static/src/elements/text.esm.js
index 610045a6..cec7a361 100644
--- a/sign_oca/static/src/elements/text.esm.js
+++ b/sign_oca/static/src/elements/text.esm.js
@@ -1,7 +1,8 @@
-/** @odoo-module **/
+/** @odoo-module QWeb **/
 
-import core from "web.core";
 import {registry} from "@web/core/registry";
+import {renderToString} from "@web/core/utils/render";
+
 const textSignOca = {
     change: function (value, parent, item) {
         item.value = value;
@@ -9,7 +10,7 @@ const textSignOca = {
     },
     generate: function (parent, item, signatureItem) {
         var input = $(
-            core.qweb.render("sign_oca.sign_iframe_field_text", {
+            renderToString("sign_oca.sign_iframe_field_text", {
                 item: item,
                 role_id: parent.info.role_id,
             })
diff --git a/sign_oca/static/src/js/main.esm.js b/sign_oca/static/src/js/main.esm.js
deleted file mode 100644
index d0e23df3..00000000
--- a/sign_oca/static/src/js/main.esm.js
+++ /dev/null
@@ -1,7 +0,0 @@
-/** @odoo-module **/
-
-import {registry} from "@web/core/registry";
-import {systrayService} from "@sign_oca/js/systray_service.esm";
-
-const serviceRegistry = registry.category("services");
-serviceRegistry.add("request_systray_service", systrayService);
diff --git a/sign_oca/static/src/js/sign_oca.esm.js b/sign_oca/static/src/js/sign_oca.esm.js
index 3498ccd6..59eeabe6 100644
--- a/sign_oca/static/src/js/sign_oca.esm.js
+++ b/sign_oca/static/src/js/sign_oca.esm.js
@@ -1,12 +1,11 @@
 /** @odoo-module */
 
-import {patch} from "@web/core/utils/patch";
-import {_t} from "web.core";
-import {useService} from "@web/core/utils/hooks";
-import {ListController} from "@web/views/list/list_controller";
 import {FormController} from "@web/views/form/form_controller";
-
+import {ListController} from "@web/views/list/list_controller";
+import {_t} from "@web/core/l10n/translation";
+import {patch} from "@web/core/utils/patch";
 const {onWillStart} = owl;
+import {useService} from "@web/core/utils/hooks";
 
 export const patchControllerSignOca = {
     setup() {
@@ -36,9 +35,8 @@ export const patchControllerSignOca = {
     },
 
     async _actionSignOcaTemplateGenerateMulti() {
-        var resIds = undefined;
-        if (this.getSelectedResIds !== undefined)
-            resIds = await this.getSelectedResIds();
+        var resIds = "";
+        if (this.getSelectedResIds) resIds = await this.getSelectedResIds();
         else resIds = this.model.root.data.id;
         this.action.doAction("sign_oca.sign_oca_template_generate_multi_act_window", {
             additionalContext: {
@@ -67,5 +65,5 @@ export const patchControllerSignOca = {
     },
 };
 
-patch(ListController.prototype, "web.ListController", patchControllerSignOca);
-patch(FormController.prototype, "web.ListController", patchControllerSignOca);
+patch(ListController.prototype, patchControllerSignOca);
+patch(FormController.prototype, patchControllerSignOca);
diff --git a/sign_oca/static/src/js/signer_menu_container.esm.js b/sign_oca/static/src/js/signer_menu_container.esm.js
index ed701f08..9aa8f207 100644
--- a/sign_oca/static/src/js/signer_menu_container.esm.js
+++ b/sign_oca/static/src/js/signer_menu_container.esm.js
@@ -1,7 +1,6 @@
 /** @odoo-module **/
 
-// ensure components are registered beforehand.
-import {getMessagingComponent} from "@mail/utils/messaging_component";
+import {registry} from "@web/core/registry";
 /* eslint-disable */
 import SignerMenuView from "./signer_menu_view.esm";
 /* eslint-enable */
@@ -22,7 +21,8 @@ export class SignerMenuContainer extends Component {
     }
 }
 
-Object.assign(SignerMenuContainer, {
-    components: {SignerMenuView: getMessagingComponent("SignerMenuView")},
-    template: "sign_oca.SignerMenuContainer",
+SignerMenuContainer.template = "sign_oca.SignerMenuContainer";
+registry.category("menu").add("SignerMenuContainer", {
+    Component: SignerMenuView,
+    props: {record: Object},
 });
diff --git a/sign_oca/static/src/js/signer_menu_view.esm.js b/sign_oca/static/src/js/signer_menu_view.esm.js
index 2009b5d5..e7b96548 100644
--- a/sign_oca/static/src/js/signer_menu_view.esm.js
+++ b/sign_oca/static/src/js/signer_menu_view.esm.js
@@ -1,8 +1,6 @@
 /** @odoo-module **/
 
-import {registerMessagingComponent} from "@mail/utils/messaging_component";
-import {useComponentToModel} from "@mail/component_hooks/use_component_to_model";
-
+import {registry} from "@web/core/registry";
 const {Component} = owl;
 
 export class SignerMenuView extends Component {
@@ -11,7 +9,6 @@ export class SignerMenuView extends Component {
      */
     setup() {
         super.setup();
-        useComponentToModel({fieldName: "component"});
     }
     /**
      * @returns {SignerMenuView}
@@ -21,9 +18,8 @@ export class SignerMenuView extends Component {
     }
 }
 
-Object.assign(SignerMenuView, {
+SignerMenuView.template = "sign_oca.SignerMenuView";
+registry.category("menu").add("SignerMenuView", {
+    Component: SignerMenuView,
     props: {record: Object},
-    template: "sign_oca.SignerMenuView",
 });
-
-registerMessagingComponent(SignerMenuView);
diff --git a/sign_oca/static/src/js/systray.esm.js b/sign_oca/static/src/js/systray.esm.js
index f2c7ba90..4d334d20 100644
--- a/sign_oca/static/src/js/systray.esm.js
+++ b/sign_oca/static/src/js/systray.esm.js
@@ -2,8 +2,7 @@
 
 import {attr, many} from "@mail/model/model_field";
 import {registerModel} from "@mail/model/model_core";
-
-import session from "web.session";
+import {session} from "@web/session";
 
 registerModel({
     name: "SignerMenuView",
diff --git a/sign_oca/static/src/js/systray_service.esm.js b/sign_oca/static/src/js/systray_service.esm.js
index 3fdce502..548640c2 100644
--- a/sign_oca/static/src/js/systray_service.esm.js
+++ b/sign_oca/static/src/js/systray_service.esm.js
@@ -15,3 +15,4 @@ export const systrayService = {
         );
     },
 };
+systrayRegistry.add("request_systray_service", systrayService);
diff --git a/sign_oca/static/src/xml/signer_menu_container.xml b/sign_oca/static/src/xml/signer_menu_container.xml
index 605d80ed..12b78e9a 100644
--- a/sign_oca/static/src/xml/signer_menu_container.xml
+++ b/sign_oca/static/src/xml/signer_menu_container.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <templates xml:space="preserve">
-    <t t-name="sign_oca.SignerMenuContainer" owl="1">
+    <t t-name="sign_oca.SignerMenuContainer">
         <t t-if="signerMenuView">
             <SignerMenuView record="signerMenuView" />
         </t>
diff --git a/sign_oca/static/src/xml/systray.xml b/sign_oca/static/src/xml/systray.xml
index e531419f..d840bab8 100644
--- a/sign_oca/static/src/xml/systray.xml
+++ b/sign_oca/static/src/xml/systray.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <templates>
-    <t t-name="sign_oca.SignerMenuView" owl="1">
+    <t t-name="sign_oca.SignerMenuView">
         <div class="o_ActivityMenuView dropdown" t-ref="root">
             <a
                 class="o_ActivityMenuView_dropdownToggle dropdown-toggle o-no-caret o-dropdown--narrow"
diff --git a/sign_oca/templates/assets.xml b/sign_oca/templates/assets.xml
index f9fe7b02..43626804 100644
--- a/sign_oca/templates/assets.xml
+++ b/sign_oca/templates/assets.xml
@@ -12,7 +12,6 @@
                 <script type="text/javascript">
                     odoo.define("sign.document_portal", function(require) {
                         var ajax = require("web.ajax");
-                        var core = require("web.core");
                         var document_signing = require("@sign_oca/components/sign_oca_pdf_portal/sign_oca_pdf_portal.esm");
                         document_signing.initDocumentToSign(<t
                         t-out="json.dumps({'access_token': access_token, 'signer_id': signer.id})"
diff --git a/sign_oca/tests/test_sign.py b/sign_oca/tests/test_sign.py
index 87fff627..dcee3db1 100644
--- a/sign_oca/tests/test_sign.py
+++ b/sign_oca/tests/test_sign.py
@@ -3,6 +3,8 @@
 
 import base64
 
+import requests
+
 from odoo.modules.module import get_module_resource
 from odoo.tests.common import Form, TransactionCase
 
@@ -10,6 +12,7 @@
 class TestSign(TransactionCase):
     @classmethod
     def setUpClass(cls):
+        cls._super_send = requests.Session.send
         super().setUpClass()
         cls.data = base64.b64encode(
             open(
@@ -83,6 +86,11 @@ def configure_request(self):
             }
         )
 
+    @classmethod
+    def _request_handler(cls, s, r, /, **kw):
+        """Don't block external requests."""
+        return cls._super_send(s, r, **kw)
+
     def test_template_configuration(self):
         self.assertFalse(self.template.get_info()["items"])
         self.configure_template()
@@ -109,6 +117,9 @@ def test_request_configuration(self):
         self.assertFalse(self.request.get_info()["items"])
         self.configure_request()
         self.assertTrue(self.request.get_info()["items"])
+        signer_info = self.request.signer_ids.get_info()
+        self.assertEqual(signer_info["partner"]["id"], self.signer.id)
+        self.assertEqual(signer_info["partner"]["name"], "Signer")
 
     def test_request_field_edition(self):
         item = self.configure_request()
@@ -214,8 +225,10 @@ def test_auto_sign_template(self):
             val = signer.get_info()["items"][key].copy()
             val["value"] = "My Name"
             data[key] = val
-        signer.action_sign(data)
+        res = signer.action_sign(data)
         self.assertEqual(signer.request_id.state, "signed")
+        self.assertEqual(res["type"], "ir.actions.act_url")
+        self.assertEqual(res["url"], signer.access_url)
 
     def test_auto_sign_template_cancel(self):
         self.configure_template()
diff --git a/sign_oca/tests/test_sign_portal.py b/sign_oca/tests/test_sign_portal.py
index 6ea712ae..9a8dcf80 100644
--- a/sign_oca/tests/test_sign_portal.py
+++ b/sign_oca/tests/test_sign_portal.py
@@ -3,41 +3,45 @@
 
 import base64
 
+import requests
+
 from odoo.modules.module import get_module_resource
 from odoo.tests.common import HttpCase, tagged
 
 
 @tagged("post_install", "-at_install")
 class TestSign(HttpCase):
-    def setUp(self):
-        super().setUp()
-        self.data = base64.b64encode(
+    @classmethod
+    def setUpClass(cls):
+        cls._super_send = requests.Session.send
+        super().setUpClass()
+        cls.data = base64.b64encode(
             open(
                 get_module_resource("sign_oca", "tests", "empty.pdf"),
                 "rb",
             ).read()
         )
-        self.signer = self.env["res.partner"].create({"name": "Signer"})
-        self.request = self.env["sign.oca.request"].create(
+        cls.signer = cls.env["res.partner"].create({"name": "Signer"})
+        cls.request = cls.env["sign.oca.request"].create(
             {
-                "data": self.data,
+                "data": cls.data,
                 "name": "Demo template",
                 "signer_ids": [
                     (
                         0,
                         0,
                         {
-                            "partner_id": self.signer.id,
-                            "role_id": self.env.ref("sign_oca.sign_role_customer").id,
+                            "partner_id": cls.signer.id,
+                            "role_id": cls.env.ref("sign_oca.sign_role_customer").id,
                         },
                     )
                 ],
             }
         )
-        self.item = self.request.add_item(
+        cls.item = cls.request.add_item(
             {
-                "role_id": self.env.ref("sign_oca.sign_role_customer").id,
-                "field_id": self.env.ref("sign_oca.sign_field_name").id,
+                "role_id": cls.env.ref("sign_oca.sign_role_customer").id,
+                "field_id": cls.env.ref("sign_oca.sign_field_name").id,
                 "page": 1,
                 "position_x": 10,
                 "position_y": 10,
@@ -46,6 +50,11 @@ def setUp(self):
             }
         )
 
+    @classmethod
+    def _request_handler(cls, s, r, /, **kw):
+        """Don't block external requests."""
+        return cls._super_send(s, r, **kw)
+
     def test_portal(self):
         self.authenticate("portal", "portal")
         self.request.action_send()
diff --git a/sign_oca/views/res_partner_views.xml b/sign_oca/views/res_partner_views.xml
index 1a7064c6..a2e4b14c 100644
--- a/sign_oca/views/res_partner_views.xml
+++ b/sign_oca/views/res_partner_views.xml
@@ -11,7 +11,7 @@
                     class="oe_stat_button"
                     type="object"
                     name="action_show_signer_ids"
-                    attrs="{'invisible': [('signer_count', '=', 0)]}"
+                    invisible="signer_count == 0"
                     icon="fa-pencil"
                     groups="sign_oca.sign_oca_group_user"
                 >
diff --git a/sign_oca/views/sign_oca_request.xml b/sign_oca/views/sign_oca_request.xml
index 337aacf4..18a5ee77 100644
--- a/sign_oca/views/sign_oca_request.xml
+++ b/sign_oca/views/sign_oca_request.xml
@@ -14,20 +14,20 @@
                         name="sign"
                         class="btn btn-primary"
                         type="object"
-                        attrs="{'invisible': ['|',('to_sign', '=', False), ('state', '!=', 'sent')]}"
+                        invisible="not to_sign or state != 'sent'"
                     />
                     <field name="to_sign" invisible="1" />
                     <button
                         string="Send"
                         type="object"
-                        states="draft"
+                        invisible="state != 'draft'"
                         name="action_send"
                         icon="fa-paper-plane"
                     />
                     <button
                         string="Cancel"
                         type="object"
-                        states="draft,sent,signed"
+                        invisible="state not in ['draft', 'sent', 'signed']"
                         name="cancel"
                         confirm="You will cancel the request and all the accesses. Are you sure about it?"
                         groups="sign_oca.sign_oca_group_user"
@@ -36,6 +36,7 @@
                         name="state"
                         widget="statusbar"
                         statusbar_visible="sent,signed"
+                        readonly="1"
                     />
                 </header>
                 <sheet>
@@ -45,7 +46,7 @@
                             type="object"
                             class="oe_stat_button"
                             icon="fa-pencil"
-                            states="sent,signed"
+                            invisible="state not in ['sent', 'signed']"
                             groups="sign_oca.sign_oca_group_user"
                         >
                             <div class="o_field_widget o_stat_info">
@@ -59,7 +60,7 @@
                         <button
                             class="oe_stat_button"
                             icon="fa-wrench"
-                            states="draft"
+                            invisible="state not in ['draft']"
                             name="configure"
                             type="object"
                             string="Configure"
@@ -78,40 +79,45 @@
                     <widget
                         name="web_ribbon"
                         title="Fully Signed"
-                        attrs="{'invisible': [('state', '!=', 'signed')]}"
+                        invisible="state != 'signed'"
                     />
                     <widget
                         name="web_ribbon"
                         bg_color="bg-danger"
                         title="Cancelled"
-                        attrs="{'invisible': [('state', '!=', 'cancel')]}"
+                        invisible="state != 'cancel'"
                     />
                     <widget
                         name="web_ribbon"
                         title="Archived"
                         bg_color="bg-danger"
-                        attrs="{'invisible': [('active', '=', True)]}"
+                        invisible="active"
                     />
                     <div class="oe_title">
                         <label for="name" />
                         <h1><field name="name" /></h1>
                     </div>
                     <group>
-                        <field name="data" filename="filename" />
+                        <field
+                            name="data"
+                            filename="filename"
+                            readonly="state != 'draft'"
+                        />
                         <field name="filename" invisible="1" />
                         <field
                             name="template_id"
-                            attrs="{'invisible': [('template_id', '=', False)]}"
+                            invisible="not template_id"
+                            readonly="1"
                         />
-                        <field name="user_id" />
-                        <field name="record_ref" />
+                        <field name="user_id" readonly="state != 'draft'" />
+                        <field name="record_ref" readonly="state != 'draft'" />
                     </group>
 
-                    <field name="signer_ids" nolabel="1">
+                    <field name="signer_ids" nolabel="1" readonly="state != 'draft'">
                         <tree editable="bottom">
                             <field name="role_id" />
                             <field name="partner_id" />
-                            <field name="signed_on" />
+                            <field name="signed_on" readonly="1" />
                         </tree>
                     </field>
                 </sheet>
@@ -222,9 +228,9 @@
                     name="sign"
                     class="btn btn-primary"
                     type="object"
-                    attrs="{'invisible': ['|',('to_sign', '=', False), ('state', '!=', 'sent')]}"
+                    invisible="not to_sign or state != 'sent'"
                 />
-                <field name="to_sign" invisible="1" />
+                <field name="to_sign" column_invisible="1" />
             </tree>
         </field>
     </record>
@@ -343,7 +349,7 @@
                         string="Sign"
                         type="object"
                         name="sign"
-                        attrs="{'invisible': [('is_allow_signature', '=', False)]}"
+                        invisible="not is_allow_signature"
                     />
                 </header>
                 <sheet>
@@ -379,7 +385,7 @@
         <field name="name">Sign Requests</field>
         <field name="res_model">sign.oca.request</field>
         <field name="view_mode">kanban,tree,form</field>
-        <field name="domain">[('template_id', '=', active_id)]</field>
+        <field name="domain">[('template_id', '=', id)]</field>
         <field name="context">{}</field>
     </record>
 
diff --git a/sign_oca/views/sign_oca_request_log.xml b/sign_oca/views/sign_oca_request_log.xml
index d9c4e9b3..6b7501c6 100644
--- a/sign_oca/views/sign_oca_request_log.xml
+++ b/sign_oca/views/sign_oca_request_log.xml
@@ -20,9 +20,9 @@
         <field name="model">sign.oca.request.log</field>
         <field name="arch" type="xml">
             <tree>
-                <field name="action" />
-                <field name="date" />
-                <field name="uid" />
+                <field name="action" readonly="1" />
+                <field name="date" readonly="1" />
+                <field name="uid" readonly="1" />
                 <field name="partner_id" />
                 <field name="signer_id" />
             </tree>
@@ -34,7 +34,7 @@
         <field name="name">Sign Requests</field>
         <field name="res_model">sign.oca.request.log</field>
         <field name="view_mode">tree</field>
-        <field name="domain">[("request_id", "=", active_id)]</field>
+        <field name="domain">[("request_id", "=", id)]</field>
         <field name="context">{}</field>
     </record>
 
diff --git a/sign_oca/views/sign_oca_role.xml b/sign_oca/views/sign_oca_role.xml
index 716b2016..765b0504 100644
--- a/sign_oca/views/sign_oca_role.xml
+++ b/sign_oca/views/sign_oca_role.xml
@@ -19,11 +19,13 @@
                             <field name="partner_selection_policy" widget="radio" />
                             <field
                                 name="default_partner_id"
-                                attrs="{'invisible': [('partner_selection_policy', '!=', 'default')], 'required': [('partner_selection_policy', '=', 'default')]}"
+                                invisible="partner_selection_policy != 'default'"
+                                required="partner_selection_policy == 'default'"
                             />
                             <field
                                 name="expression_partner"
-                                attrs="{'invisible': [('partner_selection_policy', '!=', 'expression')], 'required': [('partner_selection_policy', '=', 'expression')]}"
+                                invisible="partner_selection_policy != 'expression'"
+                                required="partner_selection_policy == 'expression'"
                             />
                         </group>
                     </group>
diff --git a/sign_oca/wizards/__init__.py b/sign_oca/wizards/__init__.py
index b173f2c6..8d660153 100644
--- a/sign_oca/wizards/__init__.py
+++ b/sign_oca/wizards/__init__.py
@@ -1,2 +1,3 @@
+from . import res_config_settings
 from . import sign_oca_template_generate
 from . import sign_oca_template_generate_multi
diff --git a/sign_oca/wizards/res_config_settings.py b/sign_oca/wizards/res_config_settings.py
new file mode 100644
index 00000000..bb023922
--- /dev/null
+++ b/sign_oca/wizards/res_config_settings.py
@@ -0,0 +1,12 @@
+# Copyright 2024 ForgeFlow S.L. (http://www.forgeflow.com)
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html).
+
+from odoo import fields, models
+
+
+class ResConfigSettings(models.TransientModel):
+    _inherit = "res.config.settings"
+
+    sign_oca_send_sign_request_copy = fields.Boolean(
+        related="company_id.sign_oca_send_sign_request_copy", readonly=False
+    )
diff --git a/sign_oca/wizards/res_config_settings_views.xml b/sign_oca/wizards/res_config_settings_views.xml
new file mode 100644
index 00000000..f972e286
--- /dev/null
+++ b/sign_oca/wizards/res_config_settings_views.xml
@@ -0,0 +1,29 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<!-- Copyright 2024 ForgeFlow S.L. (http://www.forgeflow.com)
+     License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -->
+<odoo>
+    <record id="res_config_settings_view_form" model="ir.ui.view">
+        <field name="name">res.config.settings.view.form (in sign_oca)</field>
+        <field name="model">res.config.settings</field>
+        <field name="inherit_id" ref="base.res_config_settings_view_form" />
+        <field name="arch" type="xml">
+            <xpath expr="//form" position="inside">
+                <app
+                    data-string="Sign Oca"
+                    string="Sign Oca"
+                    groups="sign_oca.sign_oca_group_admin"
+                >
+                    <block id="sign_oca_requests" title="Sign Requests">
+                        <setting
+                            class="col-xs-12 col-md-6 o_setting_box"
+                            help="Once all signers have signed the request, a copy of the final document will be sent to each of them."
+                        >
+                            <field name="sign_oca_send_sign_request_copy" />
+                            <label for="sign_oca_send_sign_request_copy" />
+                        </setting>
+                    </block>
+                </app>
+            </xpath>
+        </field>
+    </record>
+</odoo>
diff --git a/sign_oca/wizards/sign_oca_template_generate.xml b/sign_oca/wizards/sign_oca_template_generate.xml
index 473f4263..f5d0f1cd 100644
--- a/sign_oca/wizards/sign_oca_template_generate.xml
+++ b/sign_oca/wizards/sign_oca_template_generate.xml
@@ -18,10 +18,7 @@
                 <group>
                     <field name="template_id" invisible="1" />
                     <field name="sign_now" invisible="1" />
-                    <field
-                        name="message"
-                        attrs="{'invisible': [('sign_now', '=', True)]}"
-                    />
+                    <field name="message" invisible="sign_now" />
                 </group>
                 <footer>
                     <button
@@ -40,7 +37,7 @@
         <field name="name">Sign Oca Template Generate</field> <!-- TODO -->
         <field name="res_model">sign.oca.template.generate</field>
         <field name="view_mode">form</field>
-        <field name="context">{'default_template_id': active_id}</field>
+        <field name="context">{'default_template_id': id}</field>
         <field name="target">new</field>
     </record>