diff --git a/odex25_benefit/odex_benefit/models/benefit.py b/odex25_benefit/odex_benefit/models/benefit.py
index 05151d1ed..867567871 100644
--- a/odex25_benefit/odex_benefit/models/benefit.py
+++ b/odex25_benefit/odex_benefit/models/benefit.py
@@ -645,6 +645,7 @@ class GrantBenefitProfile(models.Model):
         store=True
     )
     current_rent_contract_id = fields.Many2one('rent.contract',string='Current Active Rent Contract',compute='_compute_current_rent_contract',store=True,)
+    notes = fields.Html(string='Notes', tracking=True)
 
     _sql_constraints = [
         ('unique_code', "unique (code) WHERE state NOT IN ('draft', 'new')", 'This code already exists')
diff --git a/odex25_benefit/odex_benefit/models/benefit_config.py b/odex25_benefit/odex_benefit/models/benefit_config.py
index d9984dad5..ef046bf52 100644
--- a/odex25_benefit/odex_benefit/models/benefit_config.py
+++ b/odex25_benefit/odex_benefit/models/benefit_config.py
@@ -308,6 +308,7 @@ class Salary(models.Model):
     approved = fields.Boolean(string="Is Approved", default=False)
     is_default = fields.Boolean(string='Is Default?')
     state = fields.Selection(string='Status',selection=[('waiting', 'Waiting'),('accepted', 'Accepted'),('refused', 'Refused')],default="waiting")
+    note = fields.Char(string="Note")
     # total_salary = fields.Float(string="Total Salary", compute='_compute_total_salary',store=True)
 
     # @api.depends('salary_amount','state')
diff --git a/odex25_benefit/odex_benefit/views/benefit_view.xml b/odex25_benefit/odex_benefit/views/benefit_view.xml
index 69b571dc1..b34126bf7 100644
--- a/odex25_benefit/odex_benefit/views/benefit_view.xml
+++ b/odex25_benefit/odex_benefit/views/benefit_view.xml
@@ -936,6 +936,7 @@
                                                 <field name="salary_attach" widget="many2many_attachment_preview"/>
                                                 <field name="attach_start_date"/>
                                                 <field name="attach_end_date"/>
+                                                <field name="note"/>
                                                 <field name="is_required" invisible="1"/>
                                                 <field name="is_default" invisible="1"/>
                                                 <field name="state" invisible="1"/>
@@ -1591,6 +1592,7 @@
                                         <field name="salary_attach" widget="many2many_attachment_preview"/>
                                         <field name="attach_start_date"/>
                                         <field name="attach_end_date"/>
+                                        <field name="note"/>
                                         <field name="is_required" invisible="1"/>
                                         <field name="is_default" invisible="1"/>
                                         <field name="state" invisible="1"/>
@@ -1935,6 +1937,11 @@
                                     </tree>
                                 </field>
                             </page>
+                            <page string="Notes">
+                                <group>
+                                    <field name="notes" nolabel="1"/>
+                                </group>
+                            </page>
                             <page string="Configuration" groups="odex_benefit.group_benefit_manager" invisible="1">
                                 <!--                                <group col="4">-->
                                 <!--                                    <field name="is_life" attrs="{'invisible':[('is_life','!=',False)]}"/>-->
diff --git a/odex25_benefit/odex_benefit/views/family_members.xml b/odex25_benefit/odex_benefit/views/family_members.xml
index 7fe713025..5e469331e 100644
--- a/odex25_benefit/odex_benefit/views/family_members.xml
+++ b/odex25_benefit/odex_benefit/views/family_members.xml
@@ -308,18 +308,15 @@
 
                             <page string="Attachments">
                                 <field name="attachment_ids" mode="tree" options="{'no_open': True}"
-                                       widget="one2many_list">
+                                       widget="one2many_list" attrs="{'readonly':[('state','not in',['draft','complete_info'])]}">
                                     <tree editable="bottom">
                                         <field name="name" string="Attachment Name" force_save="1"
                                                attrs="{'readonly':[('is_default','=',True)]}"/>
-                                        <field name="datas" string="File content"
-                                               attrs="{'readonly':[('parent.state','not in',['draft','complete_info'])]}"/>
+                                        <field name="datas" string="File content"/>
                                         <button name="action_preview_attachment" type="object" icon="fa-search-plus"
                                                 attrs="{'invisible': [('file_size', '=', 0)]}"/>
-                                        <field name="expiration_date"
-                                               attrs="{'readonly':[('parent.state','not in',['draft','complete_info'])]}"/>
-                                        <field name="attach_status"
-                                               attrs="{'readonly':[('parent.state','not in',['draft','complete_info'])]}"/>
+                                        <field name="expiration_date"/>
+                                        <field name="attach_status"/>
                                         <field name="allow_days"/>
                                         <field name="member_id" invisible="1"/>
                                         <field name="is_default" invisible="1"/>