From 1ed414c6fbd098ab555c187841dfc7d4c944cfe7 Mon Sep 17 00:00:00 2001
From: gsinghpal <gur_preet.singh@outlook.com>
Date: Fri, 15 May 2026 07:52:53 -0400
Subject: [PATCH] =?UTF-8?q?chore(menu):=20retire=20Configurator=20top-leve?=
 =?UTF-8?q?l=20=E2=80=94=20fold=20survivors=20into=20Configuration=20hub?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

After Phase E removed Coating Config + Treatments + Customer Price List
+ Coating Thickness from the Configurator submenu, only 3 admin items
remained — not enough to justify a top-level menu just for an
estimator.

Re-homed:
- Pricing Rules                → Configuration → Pricing & Billing
                                  (sequence 40, joins Invoice Strategy
                                   Defaults + Account Holds)
- Materials                    → Configuration → Materials & Tanks
                                  (sequence 40, joins Bath Parameters,
                                   Replenishment Rules, Chemicals,
                                   Rack Tags, Calibration Equipment)
- Line Description Templates   → Configuration → Quality & Documents
                                  (sequence 90, joins Notification
                                   Templates — same "templates" pattern)

All three keep estimator visibility (group_fp_estimator) plus manager
access. Top-level menu count under "Plating" drops from 9 visible to 8.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../__manifest__.py                           |  2 +-
 .../views/fp_configurator_menu.xml            | 26 +++++++++++--------
 .../fp_sale_description_template_views.xml    |  7 ++---
 3 files changed, 20 insertions(+), 15 deletions(-)

diff --git a/fusion_plating/fusion_plating_configurator/__manifest__.py b/fusion_plating/fusion_plating_configurator/__manifest__.py
index 31a29b44..5914d6c7 100644
--- a/fusion_plating/fusion_plating_configurator/__manifest__.py
+++ b/fusion_plating/fusion_plating_configurator/__manifest__.py
@@ -5,7 +5,7 @@
 
 {
     'name': 'Fusion Plating — Configurator',
-    'version': '19.0.20.0.0',
+    'version': '19.0.20.1.0',
     'category': 'Manufacturing/Plating',
     'summary': 'Quotation configurator with part catalog, coating configs, and formula-based pricing engine.',
     'description': """
diff --git a/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml b/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml
index aa7ebc6f..6daaf472 100644
--- a/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml
+++ b/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml
@@ -80,23 +80,27 @@
               action="action_fp_part_catalog_import_wizard"
               sequence="45"/>
 
-    <!-- ===== CONFIGURATOR submenu (admin-only: coating/pricing/treatments) ===== -->
-    <menuitem id="menu_fp_configurator"
-              name="Configurator"
-              parent="fusion_plating.menu_fp_root"
-              sequence="8"
-              groups="group_fp_estimator"/>
-
+    <!-- The Configurator top-level menu was retired in Phase F (2026-05-15)
+         after the Promote Customer Spec refactor left only 3 admin items
+         under it. They've been re-homed into the Configuration hub's
+         themed folders, where managers expect to find admin records:
+           Pricing Rules → Configuration → Pricing & Billing
+           Materials     → Configuration → Materials & Tanks
+           Line Desc Tpl → Configuration → Quality & Documents (in
+                                          fp_sale_description_template_views.xml)
+    -->
     <menuitem id="menu_fp_pricing_rules"
               name="Pricing Rules"
-              parent="menu_fp_configurator"
+              parent="fusion_plating.menu_fp_config_pricing_billing"
               action="action_fp_pricing_rule"
-              sequence="30"/>
+              sequence="40"
+              groups="group_fp_estimator,fusion_plating.group_fusion_plating_manager"/>
 
     <menuitem id="menu_fp_part_materials"
               name="Materials"
-              parent="menu_fp_configurator"
+              parent="fusion_plating.menu_fp_config_materials_tanks"
               action="action_fp_part_material"
-              sequence="50"/>
+              sequence="40"
+              groups="group_fp_estimator,fusion_plating.group_fusion_plating_manager"/>
 
 </odoo>
diff --git a/fusion_plating/fusion_plating_configurator/views/fp_sale_description_template_views.xml b/fusion_plating/fusion_plating_configurator/views/fp_sale_description_template_views.xml
index 69009803..86f7aabf 100644
--- a/fusion_plating/fusion_plating_configurator/views/fp_sale_description_template_views.xml
+++ b/fusion_plating/fusion_plating_configurator/views/fp_sale_description_template_views.xml
@@ -109,9 +109,10 @@
     </record>
 
     <menuitem id="menu_fp_sale_description_templates"
-              name="Line Descriptions"
-              parent="menu_fp_configurator"
+              name="Line Description Templates"
+              parent="fusion_plating.menu_fp_config_quality_docs"
               action="action_fp_sale_description_template"
-              sequence="45"/>
+              sequence="90"
+              groups="fusion_plating_configurator.group_fp_estimator,fusion_plating.group_fusion_plating_manager"/>
 
 </odoo>