odoo-addon-mis-builder 16.0.5.2.3.1__py3-none-any.whl → 16.0.5.4.0__py3-none-any.whl

odoo/addons/mis_builder/i18n/tr.po

@@ -16,6 +16,14 @@ msgstr ""
 "Plural-Forms: nplurals=2; plural=n != 1;\n"
 "X-Generator: Weblate 4.17\n"
 
+#. module: mis_builder
+#: model:ir.model.fields,help:mis_builder.field_mis_report_instance_annotation__annotation_context
+msgid ""
+"\n"
+"        Context used when adding annotation\n"
+"        "
+msgstr ""
+
 #. module: mis_builder
 #. odoo-python
 #: code:addons/mis_builder/models/mis_report.py:0
@@ -309,6 +317,18 @@ msgstr ""
 msgid "Analytic Domain"
 msgstr "Analitik Etki Alanı(sorgu)"
 
+#. module: mis_builder
+#. odoo-javascript
+#: code:addons/mis_builder/static/src/components/mis_report_widget.xml:0
+#, python-format
+msgid "Annotate"
+msgstr ""
+
+#. module: mis_builder
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__annotation_context
+msgid "Annotation Context"
+msgstr ""
+
 #. module: mis_builder
 #: model_terms:ir.ui.view,arch_db:mis_builder.mis_report_view_kpi_form
 msgid "Auto expand"
@@ -349,8 +369,11 @@ msgid "Bold"
 msgstr "Kalın"
 
 #. module: mis_builder
+#. odoo-javascript
+#: code:addons/mis_builder/static/src/components/mis_report_widget.esm.js:0
 #: model_terms:ir.ui.view,arch_db:mis_builder.mis_report_instance_add_to_dashboard_form_view
 #: model_terms:ir.ui.view,arch_db:mis_builder.wizard_mis_report_instance_view_form
+#, python-format
 msgid "Cancel"
 msgstr "iptal"
 
@@ -441,6 +464,7 @@ msgstr "Karşılaştırma Modu"
 #: model:ir.model.fields,field_description:mis_builder.field_add_mis_report_instance_dashboard_wizard__create_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report__create_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__create_uid
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__create_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__create_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period_sum__create_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi__create_uid
@@ -456,6 +480,7 @@ msgstr "Oluşturan"
 #: model:ir.model.fields,field_description:mis_builder.field_add_mis_report_instance_dashboard_wizard__create_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report__create_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__create_date
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__create_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__create_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period_sum__create_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi__create_date
@@ -566,6 +591,7 @@ msgstr "Sütun Açıklamalarını Görüntüle"
 #: model:ir.model.fields,field_description:mis_builder.field_add_mis_report_instance_dashboard_wizard__display_name
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report__display_name
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__display_name
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__display_name
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__display_name
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period_sum__display_name
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi__display_name
@@ -808,6 +834,7 @@ msgstr "Boş Devralmayı Gizle"
 #: model:ir.model.fields,field_description:mis_builder.field_add_mis_report_instance_dashboard_wizard__id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report__id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__id
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period_sum__id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi__id
@@ -849,6 +876,13 @@ msgstr "Girinti Düzeyi Devralma"
 msgid "Indent level must be greater than or equal to 0"
 msgstr ""
 
+#. module: mis_builder
+#. odoo-javascript
+#: code:addons/mis_builder/static/src/components/mis_report_widget.xml:0
+#, python-format
+msgid "Insert note here"
+msgstr ""
+
 #. module: mis_builder
 #: model:ir.model.fields.selection,name:mis_builder.selection__mis_report_style__font_style__italic
 msgid "Italic"
@@ -906,6 +940,7 @@ msgid "KPIs of this report and subreports."
 msgstr ""
 
 #. module: mis_builder
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__kpi_id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi_expression__kpi_id
 msgid "Kpi"
 msgstr ""
@@ -924,6 +959,7 @@ msgstr "Yatay PDF"
 #: model:ir.model.fields,field_description:mis_builder.field_add_mis_report_instance_dashboard_wizard____last_update
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report____last_update
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance____last_update
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation____last_update
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period____last_update
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period_sum____last_update
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi____last_update
@@ -945,6 +981,7 @@ msgstr "Son Oluşturulan Raporlar"
 #: model:ir.model.fields,field_description:mis_builder.field_add_mis_report_instance_dashboard_wizard__write_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report__write_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__write_uid
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__write_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__write_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period_sum__write_uid
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi__write_uid
@@ -960,6 +997,7 @@ msgstr ""
 #: model:ir.model.fields,field_description:mis_builder.field_add_mis_report_instance_dashboard_wizard__write_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report__write_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__write_date
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__write_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__write_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period_sum__write_date
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi__write_date
@@ -1066,6 +1104,16 @@ msgstr "Mali Rapor Şablonu"
 msgid "MIS Report Templates"
 msgstr "Mali Rapor Şablonları"
 
+#. module: mis_builder
+#: model:res.groups,name:mis_builder.group_edit_annotation
+msgid "MIS Report: add annotations"
+msgstr ""
+
+#. module: mis_builder
+#: model:res.groups,name:mis_builder.group_read_annotation
+msgid "MIS Report: view annotations"
+msgstr ""
+
 #. module: mis_builder
 #: model:ir.ui.menu,name:mis_builder.mis_report_conf_menu
 #: model:ir.ui.menu,name:mis_builder.mis_report_finance_menu
@@ -1104,6 +1152,11 @@ msgstr ""
 msgid "Min"
 msgstr ""
 
+#. module: mis_builder
+#: model:ir.model,name:mis_builder.model_mis_report_instance_annotation
+msgid "Mis Report Instance Annotation"
+msgstr ""
+
 #. module: mis_builder
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__mode
 msgid "Mode"
@@ -1190,6 +1243,11 @@ msgstr ""
 msgid "Normal"
 msgstr ""
 
+#. module: mis_builder
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__note
+msgid "Note"
+msgstr ""
+
 #. module: mis_builder
 #: model_terms:ir.ui.view,arch_db:mis_builder.mis_report_style_view_form
 msgid "Number"
@@ -1232,6 +1290,11 @@ msgstr ""
 msgid "Percentage"
 msgstr ""
 
+#. module: mis_builder
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__period_id
+msgid "Period"
+msgstr ""
+
 #. module: mis_builder
 #: model:ir.model.constraint,message:mis_builder.constraint_mis_report_instance_period_name_unique
 msgid "Period name should be unique by report"
@@ -1319,6 +1382,13 @@ msgstr ""
 msgid "Relative to report base date"
 msgstr ""
 
+#. module: mis_builder
+#. odoo-javascript
+#: code:addons/mis_builder/static/src/components/mis_report_widget.esm.js:0
+#, python-format
+msgid "Remove"
+msgstr ""
+
 #. module: mis_builder
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__report_id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_period__report_id
@@ -1355,8 +1425,11 @@ msgid "Rounding inherit"
 msgstr "Yuvarlama Devralma"
 
 #. module: mis_builder
+#. odoo-javascript
+#: code:addons/mis_builder/static/src/components/mis_report_widget.esm.js:0
 #: model_terms:ir.ui.view,arch_db:mis_builder.mis_report_instance_view_form
 #: model_terms:ir.ui.view,arch_db:mis_builder.wizard_mis_report_instance_view_form
+#, python-format
 msgid "Save"
 msgstr ""
 
@@ -1518,6 +1591,7 @@ msgid "Sub-KPI name ({}) must be a valid python identifier"
 msgstr ""
 
 #. module: mis_builder
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance_annotation__subkpi_id
 #: model:ir.model.fields,field_description:mis_builder.field_mis_report_kpi_expression__subkpi_id
 msgid "Subkpi"
 msgstr ""
@@ -1669,6 +1743,16 @@ msgstr ""
 msgid "Unsupported operator %s for searching on date"
 msgstr ""
 
+#. module: mis_builder
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__user_can_edit_annotation
+msgid "User Can Edit Annotation"
+msgstr ""
+
+#. module: mis_builder
+#: model:ir.model.fields,field_description:mis_builder.field_mis_report_instance__user_can_read_annotation
+msgid "User Can Read Annotation"
+msgstr ""
+
 #. module: mis_builder
 #: model:ir.actions.server,name:mis_builder.ir_cron_vacuum_temp_reports_ir_actions_server
 #: model:ir.cron,cron_name:mis_builder.ir_cron_vacuum_temp_reports
@@ -1733,6 +1817,13 @@ msgstr ""
 msgid "You cannot sum period %s with itself."
 msgstr ""
 
+#. module: mis_builder
+#. odoo-python
+#: code:addons/mis_builder/models/mis_report_instance_annotation.py:0
+#, python-format
+msgid "You do not have the rights to edit annotations"
+msgstr ""
+
 #. module: mis_builder
 #. odoo-python
 #: code:addons/mis_builder/models/aep.py:0

odoo/addons/mis_builder/models/__init__.py

@@ -8,3 +8,4 @@ from . import mis_report_style
 from . import aep
 from . import mis_kpi_data
 from . import prorata_read_group_mixin
+from . import mis_report_instance_annotation

odoo/addons/mis_builder/models/kpimatrix.py

@@ -45,10 +45,7 @@ class KpiMatrixRow:
 
     @property
     def row_id(self):
-        if not self.account_id:
-            return self.kpi.name
-        else:
-            return f"{self.kpi.name}:{self.account_id}"
+        self._matrix._make_row_id(self.kpi.id, self.account_id)
 
     def iter_cell_tuples(self, cols=None):
         if cols is None:
@@ -143,6 +140,7 @@ class KpiMatrixCell:  # noqa: B903 (immutable data class)
         self.style_props = style_props
         self.drilldown_arg = drilldown_arg
         self.val_type = val_type
+        self.cell_id = KpiMatrix._pack_cell_id(self)
 
 
 class KpiMatrix:
@@ -524,12 +522,15 @@ class KpiMatrix:
                     else:
                         val = cell.val
                     col_data = {
+                        "cell_id": cell.cell_id,
                         "val": val,
                         "val_r": cell.val_rendered,
                         "val_c": cell.val_comment,
                         "style": self._style_model.to_css_style(
                             cell.style_props, no_indent=True
                         ),
+                        # notes can not be added on 'details by account' lines
+                        "can_be_annotated": not cell.row.account_id,
                     }
                     if cell.drilldown_arg:
                         col_data["drilldown_arg"] = cell.drilldown_arg
@@ -537,3 +538,37 @@ class KpiMatrix:
             body.append(row_data)
 
         return {"header": header, "body": body}
+
+    # Logic to convert semantic coordinates (period, kpi, subkpi)
+    # to visual coordinates (cell id) and back. The rendering logic musn't know
+    # about semantic concepts such as periods and kpis. Having these well identified
+    # methods allow us to easily spot where the conversion between the rendering and
+    # semantic domain occur.
+
+    @classmethod
+    def _make_row_id(cls, kpi_id: int, account_id: int | None) -> str:
+        return f"{kpi_id}:{account_id or ''}"
+
+    @classmethod
+    def _make_cell_id(
+        cls, kpi_id: int, account_id: int | None, period_id: int, subkpi_id: int | None
+    ) -> str:
+        return f"{kpi_id}#{account_id or ''}#{period_id}#{subkpi_id or ''}"
+
+    @classmethod
+    def _pack_cell_id(cls, cell: KpiMatrixCell) -> str:
+        return cls._make_cell_id(
+            cell.row.kpi.id,
+            cell.row.account_id,
+            cell.subcol.col.key,
+            cell.subcol.subkpi and cell.subcol.subkpi.id,
+        )
+
+    @classmethod
+    def _unpack_cell_id(cls, cell_id: str) -> tuple[int, int | None, int, int | None]:
+        kpi_id, account_id, col_key, subkpi_id = cell_id.split("#")
+        kpi_id = int(kpi_id)
+        account_id = int(account_id) if account_id else None
+        period_id = int(col_key)
+        subkpi_id = int(subkpi_id) if subkpi_id else None
+        return kpi_id, account_id, period_id, subkpi_id

odoo/addons/mis_builder/models/mis_report_instance.py

@@ -13,6 +13,7 @@ from odoo.exceptions import UserError, ValidationError
 
 from .aep import AccountingExpressionProcessor as AEP
 from .expression_evaluator import ExpressionEvaluator
+from .kpimatrix import KpiMatrix
 
 _logger = logging.getLogger(__name__)
 
@@ -580,6 +581,16 @@ class MisReportInstance(models.Model):
         string="Filter box search view",
         help="Search view to customize the filter box in the report widget.",
     )
+    user_can_read_annotation = fields.Boolean(
+        compute="_compute_user_can_read_annotation",
+    )
+    user_can_edit_annotation = fields.Boolean(
+        compute="_compute_user_can_edit_annotation",
+    )
+
+    wide_display_by_default = fields.Boolean(
+        string="Open report in wide mode by default",
+    )
 
     @api.depends("report_id.move_lines_source")
     def _compute_widget_search_view_id(self):
@@ -877,7 +888,44 @@ class MisReportInstance(models.Model):
     def compute(self):
         self.ensure_one()
         kpi_matrix = self._compute_matrix()
-        return kpi_matrix.as_dict()
+        ret = kpi_matrix.as_dict()
+
+        ret["notes"] = self.get_notes_by_cell_id()
+        return ret
+
+    def get_notes_by_cell_id(self) -> dict:
+        self.ensure_one()
+        if not self.user_can_read_annotation:
+            return {}
+
+        annotations = self.env["mis.report.instance.annotation"].search(
+            [
+                ("period_id", "in", self.period_ids.ids),
+            ]
+        )
+        annotation_context = self._get_annotation_context()
+        annotations = annotations.filtered(
+            lambda rec: rec.annotation_context == annotation_context
+        )
+
+        annotations_sorted = sorted(
+            annotations,
+            key=lambda r: (
+                r.kpi_id.sequence,
+                r.period_id.sequence,
+                r.subkpi_id.sequence,
+            ),
+        )
+
+        return {
+            KpiMatrix._make_cell_id(
+                annotation.kpi_id.id,
+                False,
+                annotation.period_id.id,
+                annotation.subkpi_id and annotation.subkpi_id.id,
+            ): {"text": annotation.note, "sequence": sequence}
+            for sequence, annotation in enumerate(annotations_sorted, 1)
+        }
 
     @api.model
     def _get_drilldown_views_and_orders(self):
@@ -940,3 +988,25 @@ class MisReportInstance(models.Model):
             return f"{kpi.description} - {account.display_name} - {period.display_name}"
         else:
             return f"{kpi.description} - {period.display_name}"
+
+    def _get_annotation_context(self):
+        """Return the context used to filter annotation linked to this instance."""
+        self.ensure_one()
+        annotation_context = {}
+        if query_company_ids := self.query_company_ids.ids:
+            # sort ids to make the comparaison easier
+            annotation_context["query_company_ids"] = sorted(query_company_ids)
+
+        return annotation_context
+
+    @api.depends_context("uid")
+    def _compute_user_can_read_annotation(self):
+        self.user_can_read_annotation = self.env.user.has_group(
+            "mis_builder.group_read_annotation"
+        )
+
+    @api.depends_context("uid")
+    def _compute_user_can_edit_annotation(self):
+        self.user_can_edit_annotation = self.env.user.has_group(
+            "mis_builder.group_edit_annotation"
+        )

odoo/addons/mis_builder/models/mis_report_instance_annotation.py

@@ -0,0 +1,113 @@
+# Copyright 2025 ACSONE SA/NV
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+
+from odoo import _, api, fields, models
+from odoo.exceptions import AccessError
+
+from .kpimatrix import KpiMatrix
+
+
+class MisReportInstanceAnnotation(models.Model):
+    _name = "mis.report.instance.annotation"
+    _description = "Mis Report Instance Annotation"
+
+    period_id = fields.Many2one(
+        comodel_name="mis.report.instance.period",
+        ondelete="cascade",
+        required=True,
+    )
+    kpi_id = fields.Many2one(
+        comodel_name="mis.report.kpi",
+        ondelete="cascade",
+        required=True,
+    )
+    subkpi_id = fields.Many2one(
+        comodel_name="mis.report.subkpi",
+        ondelete="cascade",
+    )
+    note = fields.Char()
+    annotation_context = fields.Json(
+        help="""
+        Context used when adding annotation
+        """
+    )
+
+    def init(self):
+        self.env.cr.execute(
+            """
+                CREATE INDEX IF NOT EXISTS
+                mis_report_instance_annotation_period_id_kpi_id_subkpi_id_idx
+                ON mis_report_instance_annotation(period_id,kpi_id,subkpi_id);
+            """
+        )
+
+    @api.model
+    def _get_first_matching_annotation(self, cell_id, instance_id):
+        """
+        Return first annoation
+        matching exactly the period,kpi,subkpi and annotation context
+        """
+
+        kpi_id, _, period_id, subkpi_id = KpiMatrix._unpack_cell_id(cell_id)
+
+        annotations = self.env["mis.report.instance.annotation"].search(
+            [
+                ("period_id", "=", period_id),
+                ("kpi_id", "=", kpi_id),
+                ("subkpi_id", "=", subkpi_id),
+            ],
+        )
+        annotation_context = (
+            self.env["mis.report.instance"]
+            .browse(instance_id)
+            ._get_annotation_context()
+        )
+        annotation = fields.first(
+            annotations.filtered(
+                lambda rec: rec.annotation_context == annotation_context
+            )
+        )
+        return annotation
+
+    @api.model
+    def set_annotation(self, cell_id, instance_id, note):
+        if (
+            not self.env["mis.report.instance"]
+            .browse(instance_id)
+            .user_can_edit_annotation
+        ):
+            raise AccessError(_("You do not have the rights to edit annotations"))
+
+        annotation = self._get_first_matching_annotation(cell_id, instance_id)
+
+        if annotation:
+            annotation.note = note
+        else:
+            kpi_id, _account_id, period_id, subkpi_id = KpiMatrix._unpack_cell_id(
+                cell_id
+            )
+            self.env["mis.report.instance.annotation"].create(
+                {
+                    "period_id": period_id,
+                    "kpi_id": kpi_id,
+                    "subkpi_id": subkpi_id,
+                    "note": note,
+                    "annotation_context": self.env["mis.report.instance"]
+                    .browse(instance_id)
+                    ._get_annotation_context(),
+                }
+            )
+
+    @api.model
+    def remove_annotation(self, cell_id, instance_id):
+        if (
+            not self.env["mis.report.instance"]
+            .browse(instance_id)
+            .user_can_edit_annotation
+        ):
+            raise AccessError(_("You do not have the rights to edit annotations"))
+
+        annotation = self._get_first_matching_annotation(cell_id, instance_id)
+        if annotation:
+            annotation.unlink()

odoo/addons/mis_builder/report/mis_report_instance_qweb.xml

@@ -17,6 +17,7 @@
             <t t-foreach="docs" t-as="o">
                 <t t-call="web.internal_layout">
                     <t t-set="matrix" t-value="o._compute_matrix()" />
+                    <t t-set="notes" t-value="o.get_notes_by_cell_id()" />
                     <t t-set="style_obj" t-value="o.env['mis.report.style']" />
                     <div class="page">
                         <h3>
@@ -97,12 +98,34 @@
                                                 <t
                                                     t-out="cell and cell.val_rendered or ''"
                                                 />
+                                                <span
+                                                    class="oe_mis_builder_footnote"
+                                                    t-if="cell"
+                                                >
+                                                    <t
+                                                        t-out="notes.get(cell.cell_id,{}).get('sequence','')"
+                                                    />
+                                                </span>
                                             </div>
                                         </t>
                                     </div>
                                 </t>
                             </div>
                         </div>
+                        <!-- Foot notes -->
+                        <div class="oe_mis_builder_footnote_div">
+                            <table class="oe_mis_builder_footnote_table">
+                                <t
+                                    t-foreach="sorted(notes.values(),key=lambda r:r['sequence'])"
+                                    t-as="note"
+                                >
+                                    <tr>
+                                        <td><t t-out="note['sequence']" />. </td>
+                                        <td><t t-out="note['text']" /></td>
+                                    </tr>
+                                </t>
+                            </table>
+                        </div>
                     </div>
                 </t>
             </t>

odoo/addons/mis_builder/report/mis_report_instance_xlsx.py

@@ -6,7 +6,7 @@ import numbers
 from collections import defaultdict
 from datetime import datetime
 
-from odoo import _, fields, models
+from odoo import _, api, fields, models
 
 from ..models.accounting_none import AccountingNone
 from ..models.data_error import DataError
@@ -26,9 +26,18 @@ class MisBuilderXlsx(models.AbstractModel):
     _description = "MIS Builder XLSX report"
     _inherit = "report.report_xlsx.abstract"
 
+    @api.model
+    def _mis_builder_add_annotation(self, sheet, cell, row_pos, col_pos, notes):
+        """
+        Add anotation as a comment on cell in .xls
+        """
+        if cell and (annotation := notes.get(cell.cell_id, {}).get("text")):
+            sheet.write_comment(row_pos, col_pos, annotation)
+
     def generate_xlsx_report(self, workbook, data, objects):
         # get the computed result of the report
         matrix = objects._compute_matrix()
+        notes = objects.get_notes_by_cell_id()
         style_obj = self.env["mis.report.style"]
 
         # create worksheet
@@ -120,6 +129,7 @@ class MisBuilderXlsx(models.AbstractModel):
             )
             for cell in row.iter_cells():
                 col_pos += 1
+                self._mis_builder_add_annotation(sheet, cell, row_pos, col_pos, notes)
                 if not cell or cell.val is AccountingNone:
                     # TODO col/subcol format
                     sheet.write(row_pos, col_pos, "", row_format)

odoo/addons/mis_builder/security/ir.model.access.csv

@@ -20,3 +20,5 @@ access_mis_report_subreport,access_mis_report_subreport,model_mis_report_subrepo
 manage_mis_report_style,access_mis_report_style,model_mis_report_style,account.group_account_manager,1,1,1,1
 access_mis_report_style,access_mis_report_style,model_mis_report_style,base.group_user,1,0,0,0
 access_add_to_dashboard_wizard,access_add_to_dashboard_wizard,model_add_mis_report_instance_dashboard_wizard,base.group_user,1,1,1,0
+access_read_mis_report_annotation, access_read_mis_report_annotation,model_mis_report_instance_annotation,mis_builder.group_read_annotation,1,0,0,0
+access_edit_mis_report_annotation, access_edit_mis_report_annotation,model_mis_report_instance_annotation,mis_builder.group_edit_annotation,1,1,1,1

odoo/addons/mis_builder/security/res_groups.xml

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<odoo>
+    <record model="res.groups" id="group_read_annotation">
+        <field name="name">MIS Report: view annotations</field>
+    </record>
+    <record model="res.groups" id="group_edit_annotation">
+        <field name="name">MIS Report: add annotations</field>
+        <field
+            name="implied_ids"
+            eval="[Command.link(ref('mis_builder.group_read_annotation'))]"
+        />
+        <field
+            name="users"
+            eval="[(4, ref('base.user_root')), (4, ref('base.user_admin'))]"
+        />
+    </record>
+</odoo>

odoo/addons/mis_builder/static/description/index.html

@@ -367,7 +367,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:578ec6e1eb6754314b8a51c950070f53d28faebcd67c3d7014c0a53d82cca432
+!! source digest: sha256:eaf310c997ae9746cf01edf102baa7c519355121536c5a695b45470f6ab27dee
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Production/Stable" src="https://img.shields.io/badge/maturity-Production%2FStable-green.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/mis-builder/tree/16.0/mis_builder"><img alt="OCA/mis-builder" src="https://img.shields.io/badge/github-OCA%2Fmis--builder-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/mis-builder-16-0/mis-builder-16-0-mis_builder"><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/mis-builder&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 allows you to build Management Information Systems dashboards.