From 806ec5a5a6d6d37e7ee80b39f83cf9f283ceffd6 Mon Sep 17 00:00:00 2001
From: gsinghpal <gur_preet.singh@outlook.com>
Date: Sun, 31 May 2026 01:44:53 -0400
Subject: [PATCH] refactor(fusion_clock): reorganize backend menus into logical
 groups

- Attendance now groups the operational records: All Attendances, Leave
  Requests, Correction Requests, Penalties (Leaves + Penalties moved in from
  top level).
- Scheduling groups all schedule-building: Shift Planner, Scheduled Shifts,
  Shifts (templates, moved from Configuration), Schedule Audit.
- Configuration: Settings, Locations, Enroll NFC Card (the NFC wizard moved in
  from top level).
- Removed the duplicate top-level Locations menu (kept the one under Config).
Only parent/sequence changed; no actions/views touched. Live on entech 3.13.2.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 fusion_clock/__manifest__.py       |   2 +-
 fusion_clock/views/clock_menus.xml | 121 ++++++++++++++---------------
 2 files changed, 59 insertions(+), 64 deletions(-)

diff --git a/fusion_clock/__manifest__.py b/fusion_clock/__manifest__.py
index 5430134b..37f2a7bb 100644
--- a/fusion_clock/__manifest__.py
+++ b/fusion_clock/__manifest__.py
@@ -5,7 +5,7 @@
 
 {
     'name': 'Fusion Clock',
-    'version': '19.0.3.13.1',
+    'version': '19.0.3.13.2',
     'category': 'Human Resources/Attendances',
     'summary': 'Complete Employee T&A with Geofencing, Shifts, Penalties, Overtime, Kiosk, Dashboard & Payroll Export',
     'description': """
diff --git a/fusion_clock/views/clock_menus.xml b/fusion_clock/views/clock_menus.xml
index 1ad5bd87..76fafc57 100644
--- a/fusion_clock/views/clock_menus.xml
+++ b/fusion_clock/views/clock_menus.xml
@@ -32,11 +32,49 @@
               sequence="5"
               groups="group_fusion_clock_manager,group_fusion_clock_team_lead"/>
 
-    <!-- Scheduling -->
+    <!-- ============================================================
+         Attendance — day-to-day operational records
+         ============================================================ -->
+    <menuitem id="menu_fusion_clock_attendance"
+              name="Attendance"
+              parent="menu_fusion_clock_root"
+              sequence="10"/>
+
+    <menuitem id="menu_fusion_clock_attendance_list"
+              name="All Attendances"
+              parent="menu_fusion_clock_attendance"
+              action="hr_attendance.hr_attendance_action"
+              sequence="10"
+              groups="group_fusion_clock_manager,group_fusion_clock_team_lead"/>
+
+    <menuitem id="menu_fusion_clock_leaves"
+              name="Leave Requests"
+              parent="menu_fusion_clock_attendance"
+              action="action_fusion_clock_leave_request"
+              sequence="20"
+              groups="group_fusion_clock_manager"/>
+
+    <menuitem id="menu_fusion_clock_corrections"
+              name="Correction Requests"
+              parent="menu_fusion_clock_attendance"
+              action="action_fusion_clock_correction"
+              sequence="30"
+              groups="group_fusion_clock_manager"/>
+
+    <menuitem id="menu_fusion_clock_penalties"
+              name="Penalties"
+              parent="menu_fusion_clock_attendance"
+              action="action_fusion_clock_penalty"
+              sequence="40"
+              groups="group_fusion_clock_manager,group_fusion_clock_team_lead"/>
+
+    <!-- ============================================================
+         Scheduling — building and posting schedules
+         ============================================================ -->
     <menuitem id="menu_fusion_clock_scheduling"
               name="Scheduling"
               parent="menu_fusion_clock_root"
-              sequence="8"
+              sequence="20"
               groups="group_fusion_clock_manager"/>
 
     <menuitem id="menu_fusion_clock_shift_planner"
@@ -53,6 +91,13 @@
               sequence="10"
               groups="group_fusion_clock_manager"/>
 
+    <menuitem id="menu_fusion_clock_shifts"
+              name="Shifts"
+              parent="menu_fusion_clock_scheduling"
+              action="action_fusion_clock_shift"
+              sequence="15"
+              groups="group_fusion_clock_manager"/>
+
     <menuitem id="menu_fusion_clock_schedule_audit"
               name="Schedule Audit"
               parent="menu_fusion_clock_scheduling"
@@ -60,58 +105,14 @@
               sequence="20"
               groups="group_fusion_clock_manager"/>
 
-    <!-- Attendance Sub-Menu -->
-    <menuitem id="menu_fusion_clock_attendance"
-              name="Attendance"
-              parent="menu_fusion_clock_root"
-              sequence="10"/>
-
-    <menuitem id="menu_fusion_clock_attendance_list"
-              name="All Attendances"
-              parent="menu_fusion_clock_attendance"
-              action="hr_attendance.hr_attendance_action"
-              sequence="10"
-              groups="group_fusion_clock_manager,group_fusion_clock_team_lead"/>
-
-    <menuitem id="menu_fusion_clock_corrections"
-              name="Correction Requests"
-              parent="menu_fusion_clock_attendance"
-              action="action_fusion_clock_correction"
-              sequence="20"
-              groups="group_fusion_clock_manager"/>
-
-    <!-- Activity Logs -->
+    <!-- Activity Logs — cross-cutting audit trail -->
     <menuitem id="menu_fusion_clock_activity_logs"
               name="Activity Logs"
               parent="menu_fusion_clock_root"
               action="action_fusion_clock_activity_log"
-              sequence="15"
-              groups="group_fusion_clock_manager,group_fusion_clock_team_lead"/>
-
-    <!-- Locations Sub-Menu -->
-    <menuitem id="menu_fusion_clock_locations"
-              name="Locations"
-              parent="menu_fusion_clock_root"
-              action="action_fusion_clock_location"
-              sequence="20"
-              groups="group_fusion_clock_manager"/>
-
-    <!-- Penalties Sub-Menu -->
-    <menuitem id="menu_fusion_clock_penalties"
-              name="Penalties"
-              parent="menu_fusion_clock_root"
-              action="action_fusion_clock_penalty"
               sequence="30"
               groups="group_fusion_clock_manager,group_fusion_clock_team_lead"/>
 
-    <!-- Leave Requests -->
-    <menuitem id="menu_fusion_clock_leaves"
-              name="Leave Requests"
-              parent="menu_fusion_clock_root"
-              action="action_fusion_clock_leave_request"
-              sequence="35"
-              groups="group_fusion_clock_manager"/>
-
     <!-- Reports Sub-Menu -->
     <menuitem id="menu_fusion_clock_reports_parent"
               name="Reports"
@@ -141,15 +142,9 @@
               sequence="50"
               groups="group_fusion_clock_manager,group_fusion_clock_team_lead"/>
 
-    <!-- NFC Card Enrollment Wizard -->
-    <menuitem id="menu_fusion_clock_nfc_enrollment"
-              name="Enroll NFC Card"
-              parent="menu_fusion_clock_root"
-              action="action_fusion_clock_nfc_enrollment_wizard"
-              sequence="55"
-              groups="group_fusion_clock_manager"/>
-
-    <!-- Configuration Sub-Menu -->
+    <!-- ============================================================
+         Configuration — setup
+         ============================================================ -->
     <menuitem id="menu_fusion_clock_config"
               name="Configuration"
               parent="menu_fusion_clock_root"
@@ -171,13 +166,6 @@
               sequence="10"
               groups="group_fusion_clock_manager"/>
 
-    <menuitem id="menu_fusion_clock_shifts"
-              name="Shifts"
-              parent="menu_fusion_clock_config"
-              action="action_fusion_clock_shift"
-              sequence="15"
-              groups="group_fusion_clock_manager"/>
-
     <menuitem id="menu_fusion_clock_locations_config"
               name="Locations"
               parent="menu_fusion_clock_config"
@@ -185,4 +173,11 @@
               sequence="20"
               groups="group_fusion_clock_manager"/>
 
+    <menuitem id="menu_fusion_clock_nfc_enrollment"
+              name="Enroll NFC Card"
+              parent="menu_fusion_clock_config"
+              action="action_fusion_clock_nfc_enrollment_wizard"
+              sequence="30"
+              groups="group_fusion_clock_manager"/>
+
 </odoo>