From 7fdcc80bcd5fe57dc90f89856ed208281af14215 Mon Sep 17 00:00:00 2001
From: bosd <5e2fd43-d292-4c90-9d1f-74ff3436329a@anonaddy.me>
Date: Tue, 12 May 2026 21:14:17 +0200
Subject: [PATCH 1/4] [IMP] base_tier_validation: UI clean-up

Tier definition
- The form shows at the top who reviews (Four-eyes Principle next to the
  reviewer group) and in what order (narrow sequence). A Review tab
  holds the reviewer permissions and what happens when reviewing, a
  Notifications tab the notifications. These named groups are where
  modules add their options.
- Archived filter, avatar on the reviewer, name in bold, a help text on
  the empty list.
- The reminder reads "Remind every [ ] days".

Tier review
- Avatars on Requested by and Done by, the status as a badge, rows
  coloured by status.

Validation banner and systray
- Icons that match the app icon: check / xmark on Validate / Reject and
  on the validated / rejected banners, an icon on Restart Validation, and
  a check-square in the systray, which no longer looks like the sales
  and invoicing entries.
- The reviewer menu sits with the other notification menus, on the
  right of the systray.

Tier validation exceptions: name in bold.
---
 .../models/tier_definition.py                 |  6 +-
 .../models/tier_validation.py                 |  4 +-
 .../readme/newsfragments/23.feature           |  1 +
 .../static/description/index.html             |  2 +-
 .../tier_review_menu/tier_review_menu.esm.js  |  8 +-
 .../tier_review_menu/tier_review_menu.xml     |  6 +-
 .../templates/tier_validation_templates.xml   |  9 +-
 .../tests/test_tier_validation.py             | 24 ++++++
 .../views/tier_definition_view.xml            | 83 ++++++++++++++-----
 .../views/tier_review_view.xml                | 14 +++-
 .../views/tier_validation_exception_view.xml  |  2 +-
 11 files changed, 119 insertions(+), 40 deletions(-)
 create mode 100644 base_tier_validation/readme/newsfragments/23.feature

diff --git a/base_tier_validation/models/tier_definition.py b/base_tier_validation/models/tier_definition.py
index e9f3402b5..75ba13158 100644
--- a/base_tier_validation/models/tier_definition.py
+++ b/base_tier_validation/models/tier_definition.py
@@ -101,9 +101,9 @@ def _get_tier_validation_model_names(self):
         help="Default comment prefilled when approval comments are enabled.",
     )
     notify_reminder_delay = fields.Integer(
-        string="Send reminder message on pending reviews",
-        help="Number of days after which a message must be posted to remind about "
-        "pending validation  (0 = no reminder)",
+        string="Remind Every (Days)",
+        help="Remind the reviewers every this many days until the review is "
+        "done. Leave at 0 to send no reminder.",
     )
     approve_sequence = fields.Boolean(
         string="Approve by sequence",
diff --git a/base_tier_validation/models/tier_validation.py b/base_tier_validation/models/tier_validation.py
index 24de2f733..381aec44b 100644
--- a/base_tier_validation/models/tier_validation.py
+++ b/base_tier_validation/models/tier_validation.py
@@ -200,13 +200,13 @@ def _get_to_validate_message(self):
         )
 
     def _get_validated_message(self):
-        msg = f"""<i class="fa fa-thumbs-up"></i> {
+        msg = f"""<i class="fa fa-check"></i> {
             self.env._("Operation has been <b>validated</b>!")
         }"""
         return self.validation_status == "validated" and msg or ""
 
     def _get_rejected_message(self):
-        msg = f"""<i class="fa fa-thumbs-down"></i> {
+        msg = f"""<i class="fa fa-times"></i> {
             self.env._("Operation has been <b>rejected</b>.")
         }"""
         return self.validation_status == "rejected" and msg or ""
diff --git a/base_tier_validation/readme/newsfragments/23.feature b/base_tier_validation/readme/newsfragments/23.feature
new file mode 100644
index 000000000..94bda22ba
--- /dev/null
+++ b/base_tier_validation/readme/newsfragments/23.feature
@@ -0,0 +1 @@
+Clearer tier definition form: who reviews and in what order at the top, a *Review* tab (reviewer permissions, what happens when reviewing, after review) and a *Notifications* tab, where the reminder reads *Remind every [ ] days*. The definition list hides archived definitions by default, shows reviewer avatars and has an optional *Reviewer field* column. The review lists show avatars and status badges, the definition action has an empty-state help, and the reviewer menu sits with the other notification menus in the systray.
diff --git a/base_tier_validation/static/description/index.html b/base_tier_validation/static/description/index.html
index 2e9108df6..04d207354 100644
--- a/base_tier_validation/static/description/index.html
+++ b/base_tier_validation/static/description/index.html
@@ -3,7 +3,7 @@
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
 <meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
-<title>Base Tier Validation</title>
+<title>README.rst</title>
 <style type="text/css">
 
 /*
diff --git a/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.esm.js b/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.esm.js
index 40980ccd2..be9c40e0c 100644
--- a/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.esm.js
+++ b/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.esm.js
@@ -109,6 +109,12 @@ export const systrayItem = {
     Component: TierReviewMenu,
 };
 
+// Systray items render right-to-left by descending sequence (higher =
+// further left; the user menu at sequence 0 is the right-most). A low
+// sequence keeps the reviewer menu grouped with the standard
+// notification menus (messaging 25, activities 20) on the right, rather
+// than off on the far left -- notably to the right of the Enterprise AI
+// button (which sits left of the messaging menu, i.e. sequence > 25).
 registry
     .category("systray")
-    .add("base_tier_validation.ReviewerMenu", systrayItem, {sequence: 99});
+    .add("base_tier_validation.ReviewerMenu", systrayItem, {sequence: 24});
diff --git a/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.xml b/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.xml
index f4b1c323a..20582664f 100644
--- a/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.xml
+++ b/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.xml
@@ -9,11 +9,7 @@
             menuClass="discussSystray.menuClass"
         >
             <button>
-                <i
-                    class="fa fa-lg fa-pencil-square-o"
-                    role="img"
-                    aria-label="Reviews"
-                />
+                <i class="fa fa-lg fa-check-square-o" role="img" aria-label="Reviews" />
                 <span
                     t-if="store.tierReviewCounter"
                     class="o-mail-ActivityMenu-counter badge rounded-pill"
diff --git a/base_tier_validation/templates/tier_validation_templates.xml b/base_tier_validation/templates/tier_validation_templates.xml
index 288709562..01850fa13 100644
--- a/base_tier_validation/templates/tier_validation_templates.xml
+++ b/base_tier_validation/templates/tier_validation_templates.xml
@@ -14,6 +14,7 @@
                 string="Restart Validation"
                 invisible="not review_ids or hide_reviews"
                 type="object"
+                icon="fa-rotate-left"
             />
         </div>
     </template>
@@ -37,7 +38,7 @@
                         invisible="not can_review"
                         type="object"
                         class="btn-icon me-1 btn-success"
-                        icon="fa-thumbs-up"
+                        icon="fa-check fa-lg"
                     />
                     <button
                         name="reject_tier"
@@ -45,7 +46,7 @@
                         invisible="not can_review"
                         type="object"
                         class="btn-icon btn-danger"
-                        icon="fa-thumbs-down"
+                        icon="fa-times fa-lg"
                     />
                     <br />
                     <field name="next_review" readonly="1" />
@@ -57,7 +58,7 @@
                 t-attf-invisible="validation_status != 'validated' or hide_reviews or not review_ids"
             >
                 <p class="mb-1">
-                <i class="fa fa-lg fa-thumbs-up" />
+                <i class="fa fa-lg fa-check" />
                 Operation has been
                 <b>validated</b>
                 !
@@ -69,7 +70,7 @@
                 t-attf-invisible="validation_status != 'rejected' or hide_reviews or not review_ids"
             >
                 <p class="mb-1">
-                <i class="fa fa-lg fa-thumbs-down" />
+                <i class="fa fa-lg fa-times" />
                 Operation has been
                 <b>rejected</b>
                 .
diff --git a/base_tier_validation/tests/test_tier_validation.py b/base_tier_validation/tests/test_tier_validation.py
index 7e7ef8fed..92de3dc4a 100644
--- a/base_tier_validation/tests/test_tier_validation.py
+++ b/base_tier_validation/tests/test_tier_validation.py
@@ -120,6 +120,30 @@ def test_04_request_validation_rejected(self):
         record.restart_validation()
         self.assertFalse(record.review_ids)
 
+    def test_validated_and_rejected_message_icons(self):
+        """`_get_validated_message` / `_get_rejected_message` render the
+        per-status banner HTML using the check / xmark icons consistent
+        with the rest of the module's iconography. Empty until the
+        record actually reaches that state."""
+        # Until a state is reached, the banner is empty.
+        self.assertFalse(self.test_record.validated_message)
+        self.assertFalse(self.test_record.rejected_message)
+        # Validate -> banner reflects the validated state.
+        reviews = self.test_record.with_user(self.test_user_2).request_validation()
+        self.assertTrue(reviews)
+        self.test_record.with_user(self.test_user_1).validate_tier()
+        self.assertEqual(self.test_record.validation_status, "validated")
+        self.assertIn("fa-check", self.test_record.validated_message)
+        self.assertIn("validated", self.test_record.validated_message)
+        self.assertFalse(self.test_record.rejected_message)
+        # Restart + reject -> the rejected banner picks up its icon too.
+        self.test_record.restart_validation()
+        self.test_record.with_user(self.test_user_2).request_validation()
+        self.test_record.with_user(self.test_user_1).reject_tier()
+        self.assertEqual(self.test_record.validation_status, "rejected")
+        self.assertIn("fa-times", self.test_record.rejected_message)
+        self.assertIn("rejected", self.test_record.rejected_message)
+
     def test_05_under_validation(self):
         """Write is forbidden in a record under validation."""
         self.assertFalse(self.test_record.review_ids)
diff --git a/base_tier_validation/views/tier_definition_view.xml b/base_tier_validation/views/tier_definition_view.xml
index ee9513715..6892d4bfe 100644
--- a/base_tier_validation/views/tier_definition_view.xml
+++ b/base_tier_validation/views/tier_definition_view.xml
@@ -9,11 +9,11 @@
             <list>
                 <field name="sequence" widget="handle" />
                 <field name="model_id" />
-                <field name="name" />
+                <field name="name" decoration-bf="1" />
                 <field name="review_type" />
-                <field name="reviewer_id" />
+                <field name="reviewer_id" widget="many2one_avatar_user" />
                 <field name="reviewer_group_id" />
-                <field name="reviewer_field_id" />
+                <field name="reviewer_field_id" optional="hide" />
                 <field name="notify_on_create" optional="show" />
                 <field name="notify_on_pending" optional="show" />
                 <field name="has_comment" optional="hide" />
@@ -22,7 +22,6 @@
                     groups="base.group_multi_company"
                     optional="hide"
                 />
-                <field name="active" />
             </list>
         </field>
     </record>
@@ -40,7 +39,6 @@
                         invisible="active"
                     />
                     <div class="oe_title">
-                        <span class="oe_edit_only">Name</span>
                         <h1>
                             <field
                                 name="name"
@@ -54,15 +52,19 @@
                             <field name="model_id" options="{'no_create': True}" />
                             <field name="model" invisible="1" />
                             <field name="review_type" />
-                            <field name="allow_write_for_reviewer" />
                             <field
                                 name="reviewer_id"
+                                widget="many2one_avatar_user"
                                 invisible="review_type != 'individual'"
                             />
                             <field
                                 name="reviewer_group_id"
                                 invisible="review_type != 'group'"
                             />
+                            <field
+                                name="exclude_requester"
+                                invisible="review_type != 'group'"
+                            />
                             <field
                                 name="reviewer_field_id"
                                 invisible="review_type != 'field'"
@@ -75,7 +77,7 @@
                                 groups="base.group_multi_company"
                                 options="{'no_create': True}"
                             />
-                            <field name="sequence" />
+                            <field name="sequence" class="oe_inline o_input_4ch" />
                             <field name="approve_sequence" />
                             <label
                                 for="approve_sequence_bypass"
@@ -91,10 +93,6 @@
                                 <span
                                 >Bypass, if previous tier was validated by same reviewer</span>
                             </div>
-                            <field
-                                name="exclude_requester"
-                                invisible="review_type != 'group'"
-                            />
                         </group>
                     </group>
                     <notebook>
@@ -109,20 +107,42 @@
                                 />
                             </group>
                         </page>
-                        <page name="options" string="More Options">
+                        <page name="review" string="Review">
+                            <group name="review_settings">
+                                <group
+                                    string="Reviewer permissions"
+                                    name="reviewer_permissions"
+                                >
+                                    <field name="allow_write_for_reviewer" />
+                                </group>
+                                <group string="When reviewing" name="review_options">
+                                    <field name="has_comment" />
+                                    <field
+                                        name="comment_approve_default"
+                                        invisible="not has_comment"
+                                    />
+                                </group>
+                            </group>
+                        </page>
+                        <page name="options" string="Notifications">
                             <group name="more_option">
-                                <group name="notify">
+                                <group string="Notify reviewers when" name="notify">
                                     <field name="notify_on_create" />
                                     <field name="notify_on_pending" />
+                                    <label
+                                        for="notify_reminder_delay"
+                                        string="Remind every"
+                                    />
+                                    <div>
+                                        <field
+                                            name="notify_reminder_delay"
+                                            class="oe_inline o_input_4ch"
+                                        />
+                                        <span class="ms-1">days</span>
+                                    </div>
                                     <field name="notify_on_accepted" />
                                     <field name="notify_on_rejected" />
                                     <field name="notify_on_restarted" />
-                                    <field name="has_comment" />
-                                    <field
-                                        name="comment_approve_default"
-                                        invisible="not has_comment"
-                                    />
-                                    <field name="notify_reminder_delay" />
                                 </group>
                             </group>
                         </page>
@@ -142,6 +162,11 @@
                 <field name="reviewer_group_id" />
                 <field name="active" />
                 <separator />
+                <filter
+                    string="Archived"
+                    name="inactive"
+                    domain="[('active', '=', False)]"
+                />
                 <filter
                     string="All"
                     name="all"
@@ -163,7 +188,25 @@
         <field name="type">ir.actions.act_window</field>
         <field name="res_model">tier.definition</field>
         <field name="view_mode">list,form</field>
-        <field name="context">{'search_default_all': 1}</field>
+        <!-- Explicitly empty: an upgrade keeps the old show-all default
+             otherwise, and archived definitions would show up again. -->
+        <field name="context">{}</field>
+        <field name="help" type="html">
+            <p class="o_view_nocontent_smiling_face">
+                Define a new validation tier
+            </p>
+            <p>
+                A tier definition tells Odoo which records on a given model need
+                to be reviewed by which user (or group, or field-based reviewer)
+                before they can move forward in their workflow.
+            </p>
+            <p>
+                Tiers can be chained -- enable
+                <em>Approve by sequence</em>
+                on each definition for the model to require the lower-sequence
+                tier to be approved first.
+            </p>
+        </field>
     </record>
     <menuitem
         id="menu_tier_confirmation"
diff --git a/base_tier_validation/views/tier_review_view.xml b/base_tier_validation/views/tier_review_view.xml
index e0cf2509a..49ad75d02 100644
--- a/base_tier_validation/views/tier_review_view.xml
+++ b/base_tier_validation/views/tier_review_view.xml
@@ -9,14 +9,22 @@
             <list
                 decoration-danger="status=='rejected'"
                 decoration-success="status=='approved'"
+                decoration-warning="status=='pending'"
+                decoration-muted="status=='waiting'"
             >
                 <field name="sequence" />
-                <field name="requested_by" />
+                <field name="requested_by" widget="many2one_avatar_user" />
                 <field name="review_type" />
                 <field name="name" />
-                <field name="status" />
+                <field
+                    name="status"
+                    widget="badge"
+                    decoration-warning="status == 'pending'"
+                    decoration-success="status == 'approved'"
+                    decoration-danger="status == 'rejected'"
+                />
                 <field name="todo_by" />
-                <field name="done_by" />
+                <field name="done_by" widget="many2one_avatar_user" />
                 <field name="reviewed_date" />
                 <field name="comment" />
             </list>
diff --git a/base_tier_validation/views/tier_validation_exception_view.xml b/base_tier_validation/views/tier_validation_exception_view.xml
index 506e312d1..ba15bb856 100644
--- a/base_tier_validation/views/tier_validation_exception_view.xml
+++ b/base_tier_validation/views/tier_validation_exception_view.xml
@@ -7,7 +7,7 @@
         <field name="model">tier.validation.exception</field>
         <field name="arch" type="xml">
             <list>
-                <field name="name" />
+                <field name="name" decoration-bf="1" />
                 <field name="model_id" />
                 <field name="field_ids" widget="many2many_tags" optional="show" />
                 <field name="company_id" groups="base.group_multi_company" />

From e4bb6329c359544fb3a40799bc229b4c0a3ede60 Mon Sep 17 00:00:00 2001
From: bosd <5e2fd43-d292-4c90-9d1f-74ff3436329a@anonaddy.me>
Date: Sat, 3 Oct 2026 18:17:35 +0200
Subject: [PATCH 2/4] [IMP] base_tier_validation_server_action: own group in
 the Review tab

The post-approve / post-reject actions and auto-validate were appended to
the options tab of the tier definition, where they ended up on a row of
their own without a title. Give them their own "After review" group in
the new Review tab.
---
 .../views/tier_definition_view.xml            | 26 ++++++++++---------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/base_tier_validation_server_action/views/tier_definition_view.xml b/base_tier_validation_server_action/views/tier_definition_view.xml
index af37ae6b1..38f7978f8 100644
--- a/base_tier_validation_server_action/views/tier_definition_view.xml
+++ b/base_tier_validation_server_action/views/tier_definition_view.xml
@@ -7,18 +7,20 @@
         <field name="model">tier.definition</field>
         <field name="inherit_id" ref="base_tier_validation.tier_definition_view_form" />
         <field name="arch" type="xml">
-            <xpath expr="//page[@name='options']/group" position="inside">
-                <group name="server_action">
-                    <field name="server_action_id" />
-                    <field name="rejected_server_action_id" />
-                    <field name="auto_validate" />
-                    <field
-                        name="auto_validate_domain"
-                        string="if pass condition"
-                        widget="domain"
-                        options="{'model': 'model'}"
-                        invisible="not auto_validate"
-                    />
+            <xpath expr="//group[@name='review_settings']" position="after">
+                <group name="after_review">
+                    <group string="After review" name="server_action">
+                        <field name="server_action_id" />
+                        <field name="rejected_server_action_id" />
+                        <field name="auto_validate" />
+                        <field
+                            name="auto_validate_domain"
+                            string="if pass condition"
+                            widget="domain"
+                            options="{'model': 'model'}"
+                            invisible="not auto_validate"
+                        />
+                    </group>
                 </group>
             </xpath>
         </field>

From 444ed0cc6019c1ce8c41c979405a1dad68042fb5 Mon Sep 17 00:00:00 2001
From: bosd <5e2fd43-d292-4c90-9d1f-74ff3436329a@anonaddy.me>
Date: Tue, 6 Oct 2026 14:33:37 +0200
Subject: [PATCH 3/4] [IMP] base_tier_validation_forward: Allow Forward with
 the reviewer permissions

Allow Forward sat between the sequence settings at the top of the tier
definition. It is something a reviewer may do, so show it with the other
reviewer permissions in the Review tab.
---
 base_tier_validation_forward/views/tier_definition_view.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/base_tier_validation_forward/views/tier_definition_view.xml b/base_tier_validation_forward/views/tier_definition_view.xml
index dc46510f7..377c3840c 100644
--- a/base_tier_validation_forward/views/tier_definition_view.xml
+++ b/base_tier_validation_forward/views/tier_definition_view.xml
@@ -7,9 +7,9 @@
         <field name="model">tier.definition</field>
         <field name="inherit_id" ref="base_tier_validation.tier_definition_view_form" />
         <field name="arch" type="xml">
-            <field name="approve_sequence" position="after">
+            <xpath expr="//group[@name='reviewer_permissions']" position="inside">
                 <field name="has_forward" />
-            </field>
+            </xpath>
         </field>
     </record>
 </odoo>

From bf3eb1b160b700e403e5cc35ec038a56ce6b74fd Mon Sep 17 00:00:00 2001
From: bosd <5e2fd43-d292-4c90-9d1f-74ff3436329a@anonaddy.me>
Date: Tue, 6 Oct 2026 15:22:07 +0200
Subject: [PATCH 4/4] [IMP] base_tier_validation: archive Tier Validation
 Exceptions

Exceptions can now be archived instead of deleted. Archived exceptions
no longer apply. The list and search views get the usual Archived
filter, and the form groups the write permissions and the groups they
apply to, like the Tier Definition form.
---
 .../models/tier_validation.py                 |  1 +
 .../models/tier_validation_exception.py       |  1 +
 .../readme/newsfragments/23.feature           |  2 ++
 .../tests/test_tier_validation.py             | 19 +++++++++++
 .../views/tier_validation_exception_view.xml  | 33 +++++++++++++++++--
 5 files changed, 53 insertions(+), 3 deletions(-)

diff --git a/base_tier_validation/models/tier_validation.py b/base_tier_validation/models/tier_validation.py
index 381aec44b..1d6e71995 100644
--- a/base_tier_validation/models/tier_validation.py
+++ b/base_tier_validation/models/tier_validation.py
@@ -305,6 +305,7 @@ def _get_exception_fields(self, extra_domain=None):
         return (
             self.env["tier.validation.exception"]
             .sudo()
+            .with_context(active_test=True)
             .search(domain)
             .mapped("field_ids.name")
         )
diff --git a/base_tier_validation/models/tier_validation_exception.py b/base_tier_validation/models/tier_validation_exception.py
index e0c44668e..25723d390 100644
--- a/base_tier_validation/models/tier_validation_exception.py
+++ b/base_tier_validation/models/tier_validation_exception.py
@@ -19,6 +19,7 @@ def _get_tier_validation_model_names(self):
         required=True,
         default="New Tier Validation Exception",
     )
+    active = fields.Boolean(default=True)
     model_id = fields.Many2one(
         comodel_name="ir.model",
         string="Model",
diff --git a/base_tier_validation/readme/newsfragments/23.feature b/base_tier_validation/readme/newsfragments/23.feature
index 94bda22ba..cbe41db83 100644
--- a/base_tier_validation/readme/newsfragments/23.feature
+++ b/base_tier_validation/readme/newsfragments/23.feature
@@ -1 +1,3 @@
 Clearer tier definition form: who reviews and in what order at the top, a *Review* tab (reviewer permissions, what happens when reviewing, after review) and a *Notifications* tab, where the reminder reads *Remind every [ ] days*. The definition list hides archived definitions by default, shows reviewer avatars and has an optional *Reviewer field* column. The review lists show avatars and status badges, the definition action has an empty-state help, and the reviewer menu sits with the other notification menus in the systray.
+
+Tier validation exceptions can be archived; archived exceptions no longer apply. Their form groups the write permissions and the groups they apply to.
diff --git a/base_tier_validation/tests/test_tier_validation.py b/base_tier_validation/tests/test_tier_validation.py
index 92de3dc4a..e017839e9 100644
--- a/base_tier_validation/tests/test_tier_validation.py
+++ b/base_tier_validation/tests/test_tier_validation.py
@@ -1878,6 +1878,25 @@ def test_34_test_duplicate_new_user_should_not_have_review_ids(self):
         # Review_ids should not be copied when duplicating a user
         self.assertFalse(new_user.review_ids.ids)
 
+    def test_35_archived_exception_not_applied(self):
+        exception = self.env["tier.validation.exception"].create(
+            {
+                "model_id": self.tester_model.id,
+                "field_ids": [
+                    Command.set(
+                        self.tester_model.field_id.filtered(
+                            lambda f: f.name == "test_validation_field"
+                        ).ids
+                    )
+                ],
+            }
+        )
+        self.assertIn("test_validation_field", self.test_record._get_exception_fields())
+        exception.action_archive()
+        self.assertNotIn(
+            "test_validation_field", self.test_record._get_exception_fields()
+        )
+
 
 @tagged("at_install")
 class TierTierValidationView(CommonTierValidation):
diff --git a/base_tier_validation/views/tier_validation_exception_view.xml b/base_tier_validation/views/tier_validation_exception_view.xml
index ba15bb856..b7381e626 100644
--- a/base_tier_validation/views/tier_validation_exception_view.xml
+++ b/base_tier_validation/views/tier_validation_exception_view.xml
@@ -10,7 +10,11 @@
                 <field name="name" decoration-bf="1" />
                 <field name="model_id" />
                 <field name="field_ids" widget="many2many_tags" optional="show" />
-                <field name="company_id" groups="base.group_multi_company" />
+                <field
+                    name="company_id"
+                    groups="base.group_multi_company"
+                    optional="hide"
+                />
                 <field name="allowed_to_write_under_validation" />
                 <field name="allowed_to_write_after_validation" />
                 <field name="group_ids" widget="many2many_tags" />
@@ -24,6 +28,12 @@
         <field name="arch" type="xml">
             <form string="Tier Validation Exception">
                 <sheet>
+                    <widget
+                        name="web_ribbon"
+                        title="Archived"
+                        bg_color="bg-danger"
+                        invisible="active"
+                    />
                     <div class="oe_title">
                         <label for="name" />
                         <h1>
@@ -41,16 +51,27 @@
                                 widget="many2many_tags"
                                 options="{'no_create': True}"
                             />
+                        </group>
+                        <group name="right">
                             <field
                                 name="company_id"
                                 groups="base.group_multi_company"
                             />
+                            <field name="active" invisible="1" />
                         </group>
-                        <group name="right">
-                            <field name="group_ids" widget="many2many_tags" />
+                    </group>
+                    <group name="permissions">
+                        <group string="Allow editing" name="allow_write">
                             <field name="allowed_to_write_under_validation" />
                             <field name="allowed_to_write_after_validation" />
                         </group>
+                        <group string="Applies to" name="allowed_groups">
+                            <field
+                                name="group_ids"
+                                widget="many2many_tags"
+                                placeholder="All users"
+                            />
+                        </group>
                     </group>
                 </sheet>
             </form>
@@ -78,6 +99,12 @@
                     name="flt_allowed_to_write_after_validation_true"
                     domain="[('allowed_to_write_after_validation', '=', True)]"
                 />
+                <separator />
+                <filter
+                    string="Archived"
+                    name="inactive"
+                    domain="[('active', '=', False)]"
+                />
                 <group>
                     <filter
                         string="Model"