Merge pull request 'Tenant_Selection_Issue_Fixed' (#74) from Tenant_Selection_Issue_Fixed into main

Reviewed-on: #74

lib/controller/auth/login_controller.dart

@@ -14,6 +14,7 @@ class LoginController extends MyController {
   final RxBool isLoading = false.obs;
   final RxBool showPassword = false.obs;
   final RxBool isChecked = false.obs;
+  final RxBool showSplash = false.obs;
 
   @override
   void onInit() {
@@ -40,18 +41,14 @@ class LoginController extends MyController {
     );
   }
 
-  void onChangeCheckBox(bool? value) {
-    isChecked.value = value ?? false;
-  }
+  void onChangeCheckBox(bool? value) => isChecked.value = value ?? false;
 
-  void onChangeShowPassword() {
-    showPassword.toggle();
-  }
+  void onChangeShowPassword() => showPassword.toggle();
 
   Future<void> onLogin() async {
     if (!basicValidator.validateForm()) return;
 
-    isLoading.value = true;
+    showSplash.value = true; 
 
     try {
       final loginData = basicValidator.getData();
@@ -60,49 +57,30 @@ class LoginController extends MyController {
       final errors = await AuthService.loginUser(loginData);
 
       if (errors != null) {
-        logSafe(
-            "Login failed for user: ${loginData['username']} with errors: $errors",
-            level: LogLevel.warning);
-
         showAppSnackbar(
           title: "Login Failed",
           message: "Username or password is incorrect",
           type: SnackbarType.error,
         );
-
         basicValidator.addErrors(errors);
         basicValidator.validateForm();
         basicValidator.clearErrors();
       } else {
         await _handleRememberMe();
-        // ✅ Enable remote logging after successful login
         enableRemoteLogging();
-        logSafe("✅ Remote logging enabled after login.");
-
-        final fcmToken = await LocalStorage.getFcmToken();
-        if (fcmToken?.isNotEmpty ?? false) {
-          final success = await AuthService.registerDeviceToken(fcmToken!);
-          logSafe(
-              success
-                  ? "✅ FCM token registered after login."
-                  : "⚠️ Failed to register FCM token after login.",
-              level: LogLevel.warning);
-        }
-
         logSafe("Login successful for user: ${loginData['username']}");
-
-        Get.toNamed('/select_tenant');
+        Get.offNamed('/select-tenant'); 
       }
     } catch (e, stacktrace) {
-      logSafe("Exception during login",
-          level: LogLevel.error, error: e, stackTrace: stacktrace);
       showAppSnackbar(
         title: "Login Error",
         message: "An unexpected error occurred",
         type: SnackbarType.error,
       );
+      logSafe("Exception during login",
+          level: LogLevel.error, error: e, stackTrace: stacktrace);
     } finally {
-      isLoading.value = false;
+      showSplash.value = false; 
     }
   }
 
@@ -134,11 +112,7 @@ class LoginController extends MyController {
     }
   }
 
-  void goToForgotPassword() {
-    Get.toNamed('/auth/forgot_password');
-  }
+  void goToForgotPassword() => Get.toNamed('/auth/forgot_password');
 
-  void gotoRegister() {
-    Get.offAndToNamed('/auth/register_account');
-  }
+  void gotoRegister() => Get.offAndToNamed('/auth/register_account');
 }

lib/controller/auth/mpin_controller.dart

@@ -4,9 +4,10 @@ import 'package:marco/helpers/services/auth_service.dart';
 import 'package:marco/helpers/services/storage/local_storage.dart';
 import 'package:marco/helpers/widgets/my_form_validator.dart';
 import 'package:marco/helpers/widgets/my_snackbar.dart';
-import 'package:marco/view/dashboard/dashboard_screen.dart';
 import 'package:marco/helpers/services/app_logger.dart';
 import 'package:marco/helpers/services/firebase/firebase_messaging_service.dart';
+import 'package:marco/controller/permission_controller.dart';
+import 'package:marco/controller/project_controller.dart';
 
 class MPINController extends GetxController {
   final MyFormValidator basicValidator = MyFormValidator();
@@ -138,16 +139,17 @@ class MPINController extends GetxController {
   }
 
   /// Navigate to dashboard
-  void _navigateToDashboard({String? message}) {
+  /// Navigate to tenant selection after MPIN verification
+  void _navigateToTenantSelection({String? message}) {
     if (message != null) {
-      logSafe("Navigating to Dashboard with message: $message");
+      logSafe("Navigating to Tenant Selection with message: $message");
       showAppSnackbar(
         title: "Success",
         message: message,
         type: SnackbarType.success,
       );
     }
-    Get.offAll(() => const DashboardScreen());
+     Get.offAllNamed('/select-tenant');
   }
 
   /// Clear the primary MPIN fields
@@ -239,15 +241,12 @@ class MPINController extends GetxController {
     logSafe("verifyMPIN triggered");
 
     final enteredMPIN = digitControllers.map((c) => c.text).join();
-    logSafe("Entered MPIN: $enteredMPIN");
-
     if (enteredMPIN.length < 4) {
       _showError("Please enter all 4 digits.");
       return;
     }
 
     final mpinToken = await LocalStorage.getMpinToken();
-
     if (mpinToken == null || mpinToken.isEmpty) {
       _showError("Missing MPIN token. Please log in again.");
       return;
@@ -270,12 +269,25 @@ class MPINController extends GetxController {
         logSafe("MPIN verified successfully");
         await LocalStorage.setBool('mpin_verified', true);
 
+        // 🔹 Ensure controllers are injected and loaded
+        final token = await LocalStorage.getJwtToken();
+        if (token != null && token.isNotEmpty) {
+          if (!Get.isRegistered<PermissionController>()) {
+            Get.put(PermissionController());
+            await Get.find<PermissionController>().loadData(token);
+          }
+          if (!Get.isRegistered<ProjectController>()) {
+            Get.put(ProjectController(), permanent: true);
+            await Get.find<ProjectController>().fetchProjects();
+          }
+        }
+
         showAppSnackbar(
           title: "Success",
           message: "MPIN Verified Successfully",
           type: SnackbarType.success,
         );
-        _navigateToDashboard();
+        _navigateToTenantSelection();
       } else {
         final errorMessage = response["error"] ?? "Invalid MPIN";
         logSafe("MPIN verification failed: $errorMessage",
@@ -291,11 +303,7 @@ class MPINController extends GetxController {
     } catch (e) {
       isLoading.value = false;
       logSafe("Exception in verifyMPIN", level: LogLevel.error, error: e);
-      showAppSnackbar(
-        title: "Error",
-        message: "Something went wrong. Please try again.",
-        type: SnackbarType.error,
-      );
+      _showError("Something went wrong. Please try again.");
     }
   }
 

lib/controller/auth/otp_controller.dart

@@ -109,7 +109,8 @@ class OTPController extends GetxController {
   }
 
   void onOTPChanged(String value, int index) {
-    logSafe("[OTPController] OTP field changed: index=$index", level: LogLevel.debug);
+    logSafe("[OTPController] OTP field changed: index=$index",
+        level: LogLevel.debug);
     if (value.isNotEmpty) {
       if (index < otpControllers.length - 1) {
         focusNodes[index + 1].requestFocus();
@@ -125,30 +126,24 @@ class OTPController extends GetxController {
 
   Future<void> verifyOTP() async {
     final enteredOTP = otpControllers.map((c) => c.text).join();
-    logSafe("[OTPController] Verifying OTP");
-
     final result = await AuthService.verifyOtp(
       email: email.value,
       otp: enteredOTP,
     );
 
     if (result == null) {
-      logSafe("[OTPController] OTP verified successfully");
-      showAppSnackbar(
-        title: "Success",
-        message: "OTP verified successfully",
-        type: SnackbarType.success,
-      );
-      final bool isMpinEnabled = LocalStorage.getIsMpin();
-      logSafe("[OTPController] MPIN Enabled: $isMpinEnabled");
+      // ✅ Handle remember-me like in LoginController
+      final remember = LocalStorage.getBool('remember_me') ?? false;
+      if (remember) await LocalStorage.setToken('otp_email', email.value);
 
-      Get.offAllNamed('/home');
+      // ✅ Enable remote logging
+      enableRemoteLogging();
+
+      Get.offAllNamed('/select-tenant');
     } else {
-      final error = result['error'] ?? "Failed to verify OTP";
-      logSafe("[OTPController] OTP verification failed", level: LogLevel.warning, error: error);
       showAppSnackbar(
         title: "Error",
-        message: error,
+        message: result['error']!,
         type: SnackbarType.error,
       );
     }
@@ -215,7 +210,8 @@ class OTPController extends GetxController {
       final savedEmail = LocalStorage.getToken('otp_email') ?? '';
       emailController.text = savedEmail;
       email.value = savedEmail;
-      logSafe("[OTPController] Loaded saved email from local storage: $savedEmail");
+      logSafe(
+          "[OTPController] Loaded saved email from local storage: $savedEmail");
     }
   }
 }

lib/controller/auth/reset_password_controller.dart

@@ -49,8 +49,8 @@ class ResetPasswordController extends MyController {
         basicValidator.clearErrors();
       }
 
-      logSafe("[ResetPasswordController] Navigating to /home");
-      Get.toNamed('/home');
+      logSafe("[ResetPasswordController] Navigating to /dashboard");
+      Get.toNamed('/dashboard');
       update();
     } else {
       logSafe("[ResetPasswordController] Form validation failed", level: LogLevel.warning);

lib/controller/dashboard/dashboard_controller.dart

@@ -46,8 +46,9 @@ class DashboardController extends GetxController {
   // Common ranges
   final List<String> ranges = ['7D', '15D', '30D'];
 
-  // Inject ProjectController
-  final ProjectController projectController = Get.find<ProjectController>();
+// Inside your DashboardController
+  final ProjectController projectController =
+      Get.put(ProjectController(), permanent: true);
 
   @override
   void onInit() {

lib/controller/permission_controller.dart

@@ -13,6 +13,7 @@ class PermissionController extends GetxController {
   var employeeInfo = Rxn<EmployeeInfo>();
   var projectsInfo = <ProjectInfo>[].obs;
   Timer? _refreshTimer;
+  var isLoading = true.obs;
 
   @override
   void onInit() {
@@ -26,7 +27,8 @@ class PermissionController extends GetxController {
       await loadData(token!);
       _startAutoRefresh();
     } else {
-      logSafe("Token is null or empty. Skipping API load and auto-refresh.", level: LogLevel.warning);
+      logSafe("Token is null or empty. Skipping API load and auto-refresh.",
+          level: LogLevel.warning);
     }
   }
 
@@ -37,19 +39,24 @@ class PermissionController extends GetxController {
       logSafe("Auth token retrieved: $token", level: LogLevel.debug);
       return token;
     } catch (e, stacktrace) {
-      logSafe("Error retrieving auth token", level: LogLevel.error, error: e, stackTrace: stacktrace);
+      logSafe("Error retrieving auth token",
+          level: LogLevel.error, error: e, stackTrace: stacktrace);
       return null;
     }
   }
 
   Future<void> loadData(String token) async {
     try {
+      isLoading.value = true;
       final userData = await PermissionService.fetchAllUserData(token);
       _updateState(userData);
       await _storeData();
       logSafe("Data loaded and state updated successfully.");
     } catch (e, stacktrace) {
-      logSafe("Error loading data from API", level: LogLevel.error, error: e, stackTrace: stacktrace);
+      logSafe("Error loading data from API",
+          level: LogLevel.error, error: e, stackTrace: stacktrace);
+    } finally {
+      isLoading.value = false;
     }
   }
 
@@ -60,7 +67,8 @@ class PermissionController extends GetxController {
       projectsInfo.assignAll(userData['projects']);
       logSafe("State updated with user data.");
     } catch (e, stacktrace) {
-      logSafe("Error updating state", level: LogLevel.error, error: e, stackTrace: stacktrace);
+      logSafe("Error updating state",
+          level: LogLevel.error, error: e, stackTrace: stacktrace);
     }
   }
 
@@ -89,7 +97,8 @@ class PermissionController extends GetxController {
 
       logSafe("User data successfully stored in SharedPreferences.");
     } catch (e, stacktrace) {
-      logSafe("Error storing data", level: LogLevel.error, error: e, stackTrace: stacktrace);
+      logSafe("Error storing data",
+          level: LogLevel.error, error: e, stackTrace: stacktrace);
     }
   }
 
@@ -100,20 +109,23 @@ class PermissionController extends GetxController {
       if (token?.isNotEmpty ?? false) {
         await loadData(token!);
       } else {
-        logSafe("Token missing during auto-refresh. Skipping.", level: LogLevel.warning);
+        logSafe("Token missing during auto-refresh. Skipping.",
+            level: LogLevel.warning);
       }
     });
   }
 
   bool hasPermission(String permissionId) {
     final hasPerm = permissions.any((p) => p.id == permissionId);
-    logSafe("Checking permission $permissionId: $hasPerm", level: LogLevel.debug);
+    logSafe("Checking permission $permissionId: $hasPerm",
+        level: LogLevel.debug);
     return hasPerm;
   }
 
   bool isUserAssignedToProject(String projectId) {
     final assigned = projectsInfo.any((project) => project.id == projectId);
-    logSafe("Checking project assignment for $projectId: $assigned", level: LogLevel.debug);
+    logSafe("Checking project assignment for $projectId: $assigned",
+        level: LogLevel.debug);
     return assigned;
   }
 

lib/controller/task_planning/daily_task_controller.dart

@@ -78,8 +78,7 @@ class DailyTaskController extends GetxController {
     );
 
     if (response != null && response.isNotEmpty) {
-      for (var taskJson in response) {
-        final task = TaskModel.fromJson(taskJson);
+      for (var task in response) {
         final assignmentDateKey =
             task.assignmentDate.toIso8601String().split('T')[0];
         groupedDailyTasks.putIfAbsent(assignmentDateKey, () => []).add(task);

lib/controller/tenant/tenant_selection_controller.dart

@@ -4,12 +4,22 @@ import 'package:marco/helpers/services/tenant_service.dart';
 import 'package:marco/model/tenant/tenant_list_model.dart';
 import 'package:marco/helpers/widgets/my_snackbar.dart';
 import 'package:marco/helpers/services/storage/local_storage.dart';
+import 'package:marco/controller/permission_controller.dart';
 
 class TenantSelectionController extends GetxController {
   final TenantService _tenantService = TenantService();
 
-  var tenants = <Tenant>[].obs;
-  var isLoading = false.obs;
+  // Tenant list
+  final tenants = <Tenant>[].obs;
+
+  // Loading state
+  final isLoading = false.obs;
+
+  // Selected tenant ID
+  final selectedTenantId = RxnString();
+
+  // Flag to indicate auto-selection (for splash screen)
+  final isAutoSelecting = false.obs;
 
   @override
   void onInit() {
@@ -17,83 +27,97 @@ class TenantSelectionController extends GetxController {
     loadTenants();
   }
 
-  /// Load tenants from API
-  Future<void> loadTenants({bool fromTenantSelectionScreen = false}) async {
-    try {
+  /// Load tenants and handle auto-selection
+  Future<void> loadTenants() async {
     isLoading.value = true;
+    isAutoSelecting.value = true; // show splash during auto-selection
+    try {
       final data = await _tenantService.getTenants();
-      if (data != null) {
+      if (data == null || data.isEmpty) {
+        tenants.clear();
+        logSafe("⚠️ No tenants found for the user.", level: LogLevel.warning);
+        return;
+      }
+
       tenants.value = data.map((e) => Tenant.fromJson(e)).toList();
 
       final recentTenantId = LocalStorage.getRecentTenantId();
 
-        // ✅ If user came from TenantSelectionScreen & recent tenant exists, auto-select
-        if (fromTenantSelectionScreen && recentTenantId != null) {
-          final tenantExists = tenants.any((t) => t.id == recentTenantId);
-          if (tenantExists) {
-            await onTenantSelected(recentTenantId);
-            return;
-          } else {
-            // if tenant is no longer valid, clear recentTenant
-            await LocalStorage.removeRecentTenantId();
-          }
-        }
-
-        // ✅ Auto-select if only one tenant
+      // Auto-select if only one tenant
       if (tenants.length == 1) {
-          await onTenantSelected(tenants.first.id);
+        await _selectTenant(tenants.first.id);
       }
+      // Auto-select recent tenant if available
+      else if (recentTenantId != null) {
+        final recentTenant =
+            tenants.firstWhereOrNull((t) => t.id == recentTenantId);
+        if (recentTenant != null) {
+          await _selectTenant(recentTenant.id);
         } else {
-        tenants.clear();
-        logSafe("⚠️ No tenants found for the user.", level: LogLevel.warning);
+          _clearSelection();
+        }
+      }
+      // No auto-selection
+      else {
+        _clearSelection();
       }
     } catch (e, st) {
       logSafe("❌ Exception in loadTenants",
           level: LogLevel.error, error: e, stackTrace: st);
+      showAppSnackbar(
+        title: "Error",
+        message: "Failed to load organizations. Please try again.",
+        type: SnackbarType.error,
+      );
     } finally {
       isLoading.value = false;
+      isAutoSelecting.value = false; // hide splash
     }
   }
 
-  /// Select tenant
+  /// User manually selects a tenant
   Future<void> onTenantSelected(String tenantId) async {
+    isAutoSelecting.value = true;
+    await _selectTenant(tenantId);
+    isAutoSelecting.value = false;
+  }
+
+  /// Internal tenant selection logic
+  Future<void> _selectTenant(String tenantId) async {
     try {
       isLoading.value = true;
+
       final success = await _tenantService.selectTenant(tenantId);
-      if (success) {
-        logSafe("✅ Tenant selection successful: $tenantId");
-
-        // Store selected tenant in memory
-        final selectedTenant = tenants.firstWhere((t) => t.id == tenantId);
-        TenantService.setSelectedTenant(selectedTenant);
-
-        // 🔥 Save in LocalStorage
-        await LocalStorage.setRecentTenantId(tenantId);
-
-        // Navigate to dashboard
-        Get.offAllNamed('/dashboard');
-
-        showAppSnackbar(
-          title: "Success",
-          message: "Organization selected successfully.",
-          type: SnackbarType.success,
-        );
-      } else {
-        logSafe("❌ Tenant selection failed for: $tenantId",
-            level: LogLevel.warning);
-
-        // Show error snackbar
+      if (!success) {
         showAppSnackbar(
           title: "Error",
           message: "Unable to select organization. Please try again.",
           type: SnackbarType.error,
         );
+        return;
       }
-    } catch (e, st) {
-      logSafe("❌ Exception in onTenantSelected",
-          level: LogLevel.error, error: e, stackTrace: st);
 
-      // Show error snackbar for exception
+      // Update tenant & persist
+      final selectedTenant = tenants.firstWhere((t) => t.id == tenantId);
+      TenantService.setSelectedTenant(selectedTenant);
+      selectedTenantId.value = tenantId;
+      await LocalStorage.setRecentTenantId(tenantId);
+
+      // Load permissions if token exists
+      final token = LocalStorage.getJwtToken();
+      if (token != null && token.isNotEmpty) {
+        if (!Get.isRegistered<PermissionController>()) {
+          Get.put(PermissionController());
+        }
+        await Get.find<PermissionController>().loadData(token);
+      }
+
+      // Navigate **before changing isAutoSelecting**
+      await Get.offAllNamed('/dashboard');
+
+      // Then hide splash
+      isAutoSelecting.value = false;
+    } catch (e) {
       showAppSnackbar(
         title: "Error",
         message: "An unexpected error occurred while selecting organization.",
@@ -103,4 +127,10 @@ class TenantSelectionController extends GetxController {
       isLoading.value = false;
     }
   }
+
+  /// Clear tenant selection
+  void _clearSelection() {
+    selectedTenantId.value = null;
+    TenantService.currentTenant = null;
+  }
 }

lib/controller/tenant/tenant_switch_controller.dart

@@ -0,0 +1,106 @@
+import 'package:get/get.dart';
+import 'package:marco/helpers/services/app_logger.dart';
+import 'package:marco/helpers/services/tenant_service.dart';
+import 'package:marco/model/tenant/tenant_list_model.dart';
+import 'package:marco/helpers/widgets/my_snackbar.dart';
+import 'package:marco/helpers/services/storage/local_storage.dart';
+import 'package:marco/controller/permission_controller.dart';
+
+class TenantSwitchController extends GetxController {
+  final TenantService _tenantService = TenantService();
+
+  final tenants = <Tenant>[].obs;
+  final isLoading = false.obs;
+  final selectedTenantId = RxnString();
+
+  @override
+  void onInit() {
+    super.onInit();
+    loadTenants();
+  }
+
+  /// Load all tenants for switching (does not auto-select)
+  Future<void> loadTenants() async {
+    isLoading.value = true;
+    try {
+      final data = await _tenantService.getTenants();
+      if (data == null || data.isEmpty) {
+        tenants.clear();
+        logSafe("⚠️ No tenants available for switching.", level: LogLevel.warning);
+        return;
+      }
+
+      tenants.value = data.map((e) => Tenant.fromJson(e)).toList();
+
+      // Keep current tenant as selected
+      selectedTenantId.value = TenantService.currentTenant?.id;
+    } catch (e, st) {
+      logSafe("❌ Exception in loadTenants", level: LogLevel.error, error: e, stackTrace: st);
+      showAppSnackbar(
+        title: "Error",
+        message: "Failed to load organizations for switching.",
+        type: SnackbarType.error,
+      );
+    } finally {
+      isLoading.value = false;
+    }
+  }
+
+  /// Switch to a different tenant and navigate fully
+  Future<void> switchTenant(String tenantId) async {
+    if (TenantService.currentTenant?.id == tenantId) return; 
+
+    isLoading.value = true;
+    try {
+      final success = await _tenantService.selectTenant(tenantId);
+      if (!success) {
+        logSafe("❌ Tenant switch failed: $tenantId", level: LogLevel.warning);
+        showAppSnackbar(
+          title: "Error",
+          message: "Unable to switch organization. Try again.",
+          type: SnackbarType.error,
+        );
+        return;
+      }
+
+      final selectedTenant = tenants.firstWhere((t) => t.id == tenantId);
+      TenantService.setSelectedTenant(selectedTenant);
+      selectedTenantId.value = tenantId;
+
+      // Persist recent tenant
+      await LocalStorage.setRecentTenantId(tenantId);
+
+      logSafe("✅ Tenant switched successfully: $tenantId");
+
+      // 🔹 Load permissions after tenant switch (null-safe)
+      final token = await LocalStorage.getJwtToken();
+      if (token != null && token.isNotEmpty) {
+        if (!Get.isRegistered<PermissionController>()) {
+          Get.put(PermissionController());
+          logSafe("✅ PermissionController injected after tenant switch.");
+        }
+        await Get.find<PermissionController>().loadData(token);
+      } else {
+        logSafe("⚠️ JWT token is null. Cannot load permissions.", level: LogLevel.warning);
+      }
+
+      // FULL NAVIGATION: reload app/dashboard
+      Get.offAllNamed('/dashboard');
+
+      showAppSnackbar(
+        title: "Success",
+        message: "Switched to organization: ${selectedTenant.name}",
+        type: SnackbarType.success,
+      );
+    } catch (e, st) {
+      logSafe("❌ Exception in switchTenant", level: LogLevel.error, error: e, stackTrace: st);
+      showAppSnackbar(
+        title: "Error",
+        message: "An unexpected error occurred while switching organization.",
+        type: SnackbarType.error,
+      );
+    } finally {
+      isLoading.value = false;
+    }
+  }
+}

lib/helpers/services/api_service.dart

@@ -20,6 +20,7 @@ import 'package:marco/model/document/document_details_model.dart';
 import 'package:marco/model/document/document_version_model.dart';
 import 'package:marco/model/attendance/organization_per_project_list_model.dart';
 import 'package:marco/model/tenant/tenant_services_model.dart';
+import 'package:marco/model/dailyTaskPlanning/daily_task_model.dart';
 
 class ApiService {
   static const bool enableLogs = true;
@@ -354,14 +355,24 @@ class ApiService {
     logSafe("Posting logs... count=${logs.length}");
 
     try {
-      final response =
-          await _postRequest(endpoint, logs, customTimeout: extendedTimeout);
-
-      if (response == null) {
-        logSafe("Post logs failed: null response", level: LogLevel.error);
+      // Get token directly without triggering logout or refresh
+      final token = await LocalStorage.getJwtToken();
+      if (token == null) {
+        logSafe("No token available. Skipping logs post.",
+            level: LogLevel.warning);
         return false;
       }
 
+      final uri = Uri.parse("${ApiEndpoints.baseUrl}$endpoint");
+      final headers = {
+        'Content-Type': 'application/json',
+        'Authorization': 'Bearer $token',
+      };
+
+      final response = await http
+          .post(uri, headers: headers, body: jsonEncode(logs))
+          .timeout(ApiService.extendedTimeout);
+
       logSafe("Post logs response status: ${response.statusCode}");
       logSafe("Post logs response body: ${response.body}");
 
@@ -1761,10 +1772,10 @@ class ApiService {
     return false;
   }
 
-static Future<List<dynamic>?> getDirectoryComments(
+  static Future<List<dynamic>?> getDirectoryComments(
     String contactId, {
     bool active = true,
-}) async {
+  }) async {
     final url = "${ApiEndpoints.getDirectoryNotes}/$contactId?active=$active";
     final response = await _getRequest(url);
     final data = response != null
@@ -1772,8 +1783,7 @@ static Future<List<dynamic>?> getDirectoryComments(
         : null;
 
     return data is List ? data : null;
-}
-
+  }
 
   static Future<bool> updateContact(
       String contactId, Map<String, dynamic> payload) async {
@@ -2116,14 +2126,14 @@ static Future<List<dynamic>?> getDirectoryComments(
 
   // === Daily Task APIs ===
 
-  static Future<List<dynamic>?> getDailyTasks(
+  static Future<List<TaskModel>?> getDailyTasks(
   String projectId, {
   DateTime? dateFrom,
   DateTime? dateTo,
   List<String>? serviceIds,
   int pageNumber = 1,
   int pageSize = 20,
-  }) async {
+}) async {
   final filterBody = {
     "serviceIds": serviceIds ?? [],
   };
@@ -2142,12 +2152,16 @@ static Future<List<dynamic>?> getDirectoryComments(
       Uri.parse(ApiEndpoints.getDailyTask).replace(queryParameters: query);
 
   final response = await _getRequest(uri.toString());
+  final parsed = response != null ? _parseResponse(response, label: 'Daily Tasks') : null;
 
-    return response != null
-        ? _parseResponse(response, label: 'Daily Tasks')
-        : null;
+  if (parsed != null && parsed['data'] != null) {
+    return (parsed['data'] as List).map((e) => TaskModel.fromJson(e)).toList();
   }
 
+  return null;
+}
+
+
   static Future<bool> reportTask({
     required String id,
     required int completedTask,

lib/helpers/services/app_initializer.dart

@@ -1,10 +1,6 @@
 import 'package:flutter/services.dart';
-import 'package:get/get.dart';
 import 'package:url_strategy/url_strategy.dart';
 import 'package:firebase_core/firebase_core.dart';
-
-import 'package:marco/controller/permission_controller.dart';
-import 'package:marco/controller/project_controller.dart';
 import 'package:marco/helpers/services/storage/local_storage.dart';
 import 'package:marco/helpers/services/app_logger.dart';
 import 'package:marco/helpers/services/auth_service.dart';
@@ -26,7 +22,6 @@ Future<void> initializeApp() async {
     await _setupDeviceInfo();
     await _handleAuthTokens(); 
     await _setupTheme();
-    await _setupControllers();
     await _setupFirebaseMessaging();
 
     _finalizeAppStyle();
@@ -43,6 +38,19 @@ Future<void> initializeApp() async {
   }
 }
 
+Future<void> _handleAuthTokens() async {
+  final refreshToken = await LocalStorage.getRefreshToken();
+  if (refreshToken?.isNotEmpty ?? false) {
+    logSafe("🔁 Refresh token found. Attempting to refresh JWT...");
+    final success = await AuthService.refreshToken();
+    if (!success) {
+      logSafe("⚠️ Refresh token invalid or expired. User must login again.");
+    }
+  } else {
+    logSafe("❌ No refresh token found. Skipping refresh.");
+  }
+}
+
 Future<void> _setupUI() async {
   setPathUrlStrategy();
   await SystemChrome.setEnabledSystemUIMode(SystemUiMode.edgeToEdge);
@@ -69,50 +77,11 @@ Future<void> _setupDeviceInfo() async {
   logSafe("📱 Device Info: ${deviceInfoService.deviceData}");
 }
 
-Future<void> _handleAuthTokens() async {
-  final refreshToken = await LocalStorage.getRefreshToken();
-  if (refreshToken?.isNotEmpty ?? false) {
-    logSafe("🔁 Refresh token found. Attempting to refresh JWT...");
-    final success = await AuthService.refreshToken();
-    if (!success) {
-      logSafe(
-          "⚠️ Refresh token invalid or expired. Skipping controller injection.");
-    }
-  } else {
-    logSafe("❌ No refresh token found. Skipping refresh.");
-  }
-}
-
 Future<void> _setupTheme() async {
   await ThemeCustomizer.init();
   logSafe("💡 Theme customizer initialized.");
 }
 
-Future<void> _setupControllers() async {
-  final token = LocalStorage.getString('jwt_token');
-  if (token?.isEmpty ?? true) {
-    logSafe("⚠️ No valid JWT token found. Skipping controller initialization.");
-    return;
-  }
-
-  if (!Get.isRegistered<PermissionController>()) {
-    Get.put(PermissionController());
-    logSafe("💡 PermissionController injected.");
-  }
-
-  if (!Get.isRegistered<ProjectController>()) {
-    Get.put(ProjectController(), permanent: true);
-    logSafe("💡 ProjectController injected as permanent.");
-  }
-
-  await Future.wait([
-    Get.find<PermissionController>().loadData(token!),
-    Get.find<ProjectController>().fetchProjects(),
-  ]);
-}
-
-// ❌ Commented out Firebase Messaging setup
-
 Future<void> _setupFirebaseMessaging() async {
   await FirebaseNotificationService().initialize();
   logSafe("💡 Firebase Messaging initialized.");

lib/helpers/services/auth_service.dart

@@ -1,9 +1,5 @@
 import 'dart:convert';
-import 'package:get/get.dart';
 import 'package:http/http.dart' as http;
-
-import 'package:marco/controller/permission_controller.dart';
-import 'package:marco/controller/project_controller.dart';
 import 'package:marco/helpers/services/api_endpoints.dart';
 import 'package:marco/helpers/services/storage/local_storage.dart';
 import 'package:marco/helpers/services/app_logger.dart';
@@ -98,8 +94,8 @@ class AuthService {
   }
 
   static Future<bool> refreshToken() async {
-    final accessToken = await LocalStorage.getJwtToken();
-    final refreshToken = await LocalStorage.getRefreshToken();
+    final accessToken =  LocalStorage.getJwtToken();
+    final refreshToken =  LocalStorage.getRefreshToken();
 
     if ([accessToken, refreshToken].any((t) => t == null || t.isEmpty)) {
       logSafe("Missing access or refresh token.", level: LogLevel.warning);
@@ -115,7 +111,7 @@ class AuthService {
       logSafe("Token refreshed successfully.");
 
       // 🔹 Retry FCM token registration after token refresh
-      final newFcmToken = await LocalStorage.getFcmToken();
+      final newFcmToken =  LocalStorage.getFcmToken();
       if (newFcmToken?.isNotEmpty ?? false) {
         final success = await registerDeviceToken(newFcmToken!);
         logSafe(
@@ -157,7 +153,7 @@ class AuthService {
   }) =>
       _wrapErrorHandling(
         () async {
-          final token = await LocalStorage.getJwtToken();
+          final token =  LocalStorage.getJwtToken();
           return _post(
             "/auth/generate-mpin",
             {"employeeId": employeeId, "mpin": mpin},
@@ -290,30 +286,6 @@ class AuthService {
       await LocalStorage.setIsMpin(false);
       await LocalStorage.removeMpinToken();
     }
-
-    if (!Get.isRegistered<PermissionController>()) {
-      Get.put(PermissionController());
-      logSafe("✅ PermissionController injected after login.");
-    }
-    if (!Get.isRegistered<ProjectController>()) {
-      Get.put(ProjectController(), permanent: true);
-      logSafe("✅ ProjectController injected after login.");
-    }
-
-    await Get.find<PermissionController>().loadData(data['token']);
-    await Get.find<ProjectController>().fetchProjects();
-
-    // 🔹 Always try to register FCM token after login
-    final fcmToken = await LocalStorage.getFcmToken();
-    if (fcmToken?.isNotEmpty ?? false) {
-      final success = await registerDeviceToken(fcmToken!);
-      logSafe(
-          success
-              ? "✅ FCM token registered after login."
-              : "⚠️ Failed to register FCM token after login.",
-          level: success ? LogLevel.info : LogLevel.warning);
-    }
-
     isLoggedIn = true;
     logSafe("✅ Login flow completed and controllers initialized.");
   }

lib/helpers/services/firebase/firebase_messaging_service.dart

@@ -19,7 +19,7 @@ class FirebaseNotificationService {
     _registerMessageListeners();
     _registerTokenRefreshListener();
 
-    // Fetch token on app start (but only register with server if JWT available)
+    // Fetch token on app start (and register with server if JWT available)
     await getFcmToken(registerOnServer: true);
   }
 
@@ -49,6 +49,7 @@ class FirebaseNotificationService {
 
     FirebaseMessaging.onMessageOpenedApp.listen(_handleNotificationTap);
 
+    // Background messages
     FirebaseMessaging.onBackgroundMessage(_firebaseMessagingBackgroundHandler);
   }
 
@@ -111,8 +112,6 @@ class FirebaseNotificationService {
     }
   }
 
- 
- 
   /// Handle tap on notification
   void _handleNotificationTap(RemoteMessage message) {
     _logger.i('📌 Notification tapped: ${message.data}');
@@ -129,7 +128,9 @@ class FirebaseNotificationService {
   }
 }
 
-/// Background handler (required by Firebase)
+/// 🔹 Background handler (required by Firebase)
+/// Must be a top-level function and annotated for AOT
+@pragma('vm:entry-point')
 Future<void> _firebaseMessagingBackgroundHandler(RemoteMessage message) async {
   final logger = Logger();
   logger

lib/helpers/services/tenant_service.dart

@@ -129,6 +129,17 @@ class TenantService implements ITenantService {
           logSafe("⚠️ ProjectController not found while refreshing projects");
         }
 
+        // 🔹 Register FCM token after tenant selection
+        final fcmToken =  LocalStorage.getFcmToken();
+        if (fcmToken?.isNotEmpty ?? false) {
+          final success = await AuthService.registerDeviceToken(fcmToken!);
+          logSafe(
+              success
+                  ? "✅ FCM token registered after tenant selection."
+                  : "⚠️ Failed to register FCM token after tenant selection.",
+              level: success ? LogLevel.info : LogLevel.warning);
+        }
+
         return true;
       }
 

lib/helpers/theme/app_theme.dart

@@ -230,7 +230,7 @@ class AppStyle {
       containerRadius: AppStyle.containerRadius.medium,
       cardRadius: AppStyle.cardRadius.medium,
       buttonRadius: AppStyle.buttonRadius.medium,
-      defaultBreadCrumbItem: MyBreadcrumbItem(name: 'Marco', route: '/client/home'),
+      defaultBreadCrumbItem: MyBreadcrumbItem(name: 'Marco', route: '/client/dashboard'),
     ));
     bool isMobile = true;
     try {

lib/helpers/widgets/dashbaord/attendance_overview_chart.dart

@@ -60,7 +60,6 @@ class AttendanceDashboardChart extends StatelessWidget {
 
       final filteredData = _getFilteredData();
 
-
       return Container(
         decoration: _containerDecoration,
         padding: EdgeInsets.symmetric(
@@ -254,7 +253,7 @@ class _AttendanceChart extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    final dateFormat = DateFormat('d MMMM');
+    final dateFormat = DateFormat('d MMM');
     final uniqueDates = data
         .map((e) => DateTime.parse(e['date'] as String))
         .toSet()
@@ -273,10 +272,6 @@ class _AttendanceChart extends StatelessWidget {
     if (allZero) {
       return Container(
         height: 600,
-        decoration: BoxDecoration(
-          color: Colors.blueGrey.shade50,
-          borderRadius: BorderRadius.circular(5),
-        ),
         child: const Center(
           child: Text(
             'No attendance data for the selected range.',
@@ -302,7 +297,6 @@ class _AttendanceChart extends StatelessWidget {
       height: 600,
       padding: const EdgeInsets.all(6),
       decoration: BoxDecoration(
-        color: Colors.blueGrey.shade50,
         borderRadius: BorderRadius.circular(5),
       ),
       child: SfCartesianChart(
@@ -317,7 +311,7 @@ class _AttendanceChart extends StatelessWidget {
                 return {'date': date, 'present': formattedMap[key] ?? 0};
               })
               .where((d) => (d['present'] ?? 0) > 0)
-              .toList(); // ✅ remove 0 bars
+              .toList();
 
           return StackedColumnSeries<Map<String, dynamic>, String>(
             dataSource: seriesData,
@@ -358,7 +352,7 @@ class _AttendanceTable extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    final dateFormat = DateFormat('d MMMM');
+    final dateFormat = DateFormat('d MMM');
     final uniqueDates = data
         .map((e) => DateTime.parse(e['date'] as String))
         .toSet()
@@ -377,10 +371,6 @@ class _AttendanceTable extends StatelessWidget {
     if (allZero) {
       return Container(
         height: 300,
-        decoration: BoxDecoration(
-          color: Colors.grey.shade50,
-          borderRadius: BorderRadius.circular(5),
-        ),
         child: const Center(
           child: Text(
             'No attendance data for the selected range.',
@@ -402,15 +392,22 @@ class _AttendanceTable extends StatelessWidget {
       decoration: BoxDecoration(
         border: Border.all(color: Colors.grey.shade300),
         borderRadius: BorderRadius.circular(5),
-        color: Colors.grey.shade50,
       ),
+      child: Scrollbar(
+        thumbVisibility: true,
+        trackVisibility: true,
         child: SingleChildScrollView(
           scrollDirection: Axis.horizontal,
+          child: ConstrainedBox(
+            constraints:
+                BoxConstraints(minWidth: MediaQuery.of(context).size.width),
+            child: SingleChildScrollView(
+              scrollDirection: Axis.vertical,
               child: DataTable(
-          columnSpacing: screenWidth < 600 ? 20 : 36,
+                columnSpacing: 20,
                 headingRowHeight: 44,
-          headingRowColor:
-              MaterialStateProperty.all(Colors.blueAccent.withOpacity(0.08)),
+                headingRowColor: MaterialStateProperty.all(
+                    Colors.blueAccent.withOpacity(0.08)),
                 headingTextStyle: const TextStyle(
                     fontWeight: FontWeight.bold, color: Colors.black87),
                 columns: [
@@ -420,7 +417,8 @@ class _AttendanceTable extends StatelessWidget {
                 rows: filteredRoles.map((role) {
                   return DataRow(
                     cells: [
-                DataCell(_RolePill(role: role, color: getRoleColor(role))),
+                      DataCell(
+                          _RolePill(role: role, color: getRoleColor(role))),
                       ...filteredDates.map((date) {
                         final key = '${role}_$date';
                         return DataCell(
@@ -436,6 +434,9 @@ class _AttendanceTable extends StatelessWidget {
                 }).toList(),
               ),
             ),
+          ),
+        ),
+      ),
     );
   }
 }

lib/helpers/widgets/dashbaord/project_progress_chart.dart

@@ -197,13 +197,13 @@ class ProjectProgressChart extends StatelessWidget {
       height: height > 280 ? 280 : height,
       padding: const EdgeInsets.all(6),
       decoration: BoxDecoration(
-        color: Colors.blueGrey.shade50,
+        // Remove background
+        color: Colors.transparent,
         borderRadius: BorderRadius.circular(5),
       ),
       child: SfCartesianChart(
         tooltipBehavior: TooltipBehavior(enable: true),
         legend: Legend(isVisible: true, position: LegendPosition.bottom),
-        // ✅ Use CategoryAxis so only nonZeroData dates show up
         primaryXAxis: CategoryAxis(
           majorGridLines: const MajorGridLines(width: 0),
           axisLine: const AxisLine(width: 0),
@@ -273,14 +273,15 @@ class ProjectProgressChart extends StatelessWidget {
       decoration: BoxDecoration(
         border: Border.all(color: Colors.grey.shade300),
         borderRadius: BorderRadius.circular(5),
-        color: Colors.grey.shade50,
+        color: Colors.transparent,
       ),
-      child: LayoutBuilder(
-        builder: (context, constraints) {
-          return SingleChildScrollView(
+      child: Scrollbar(
+        thumbVisibility: true,
+        trackVisibility: true,
+        child: SingleChildScrollView(
           scrollDirection: Axis.horizontal,
           child: ConstrainedBox(
-              constraints: BoxConstraints(minWidth: constraints.maxWidth),
+            constraints: BoxConstraints(minWidth: screenWidth),
             child: SingleChildScrollView(
               scrollDirection: Axis.vertical,
               child: DataTable(
@@ -299,22 +300,17 @@ class ProjectProgressChart extends StatelessWidget {
                   return DataRow(
                     cells: [
                       DataCell(Text(DateFormat('d MMM').format(task.date))),
-                        DataCell(Text(
-                          '${task.planned}',
-                          style: TextStyle(color: _getTaskColor('Planned')),
-                        )),
-                        DataCell(Text(
-                          '${task.completed}',
-                          style: TextStyle(color: _getTaskColor('Completed')),
-                        )),
+                      DataCell(Text('${task.planned}',
+                          style: TextStyle(color: _getTaskColor('Planned')))),
+                      DataCell(Text('${task.completed}',
+                          style: TextStyle(color: _getTaskColor('Completed')))),
                     ],
                   );
                 }).toList(),
               ),
             ),
           ),
-          );
-        },
+        ),
       ),
     );
   }
@@ -323,7 +319,7 @@ class ProjectProgressChart extends StatelessWidget {
     return Container(
       height: height > 280 ? 280 : height,
       decoration: BoxDecoration(
-        color: Colors.blueGrey.shade50,
+        color: Colors.transparent,
         borderRadius: BorderRadius.circular(5),
       ),
       child: const Center(

lib/helpers/widgets/tenant/organization_selector.dart

@@ -77,7 +77,32 @@ class OrganizationSelector extends StatelessWidget {
   Widget build(BuildContext context) {
     return Obx(() {
       if (controller.isLoadingOrganizations.value) {
-        return const Center(child: CircularProgressIndicator());
+        return Container(
+          height: height ?? 40,
+          padding: const EdgeInsets.symmetric(horizontal: 12),
+          decoration: BoxDecoration(
+            color: Colors.grey.shade300,
+            borderRadius: BorderRadius.circular(12),
+          ),
+          child: Row(
+            mainAxisAlignment: MainAxisAlignment.spaceBetween,
+            children: [
+              Container(
+                width: 100,
+                height: 14,
+                color: Colors.grey.shade400,
+              ),
+              Container(
+                width: 18,
+                height: 18,
+                decoration: BoxDecoration(
+                  color: Colors.grey.shade400,
+                  shape: BoxShape.circle,
+                ),
+              ),
+            ],
+          ),
+        );
       } else if (controller.organizations.isEmpty) {
         return Center(
           child: Padding(
@@ -96,7 +121,6 @@ class OrganizationSelector extends StatelessWidget {
         ...controller.organizations.map((e) => e.name)
       ];
 
-      // Listen to selectedOrganization.value
       return _popupSelector(
         currentValue: controller.currentSelection,
         items: orgNames,

lib/helpers/widgets/tenant/service_selector.dart

@@ -88,11 +88,40 @@ class ServiceSelector extends StatelessWidget {
     );
   }
 
+  Widget _skeletonSelector() {
+    return Container(
+      height: height ?? 40,
+      padding: const EdgeInsets.symmetric(horizontal: 12),
+      decoration: BoxDecoration(
+        color: Colors.grey.shade300,
+        borderRadius: BorderRadius.circular(10),
+      ),
+      child: Row(
+        mainAxisAlignment: MainAxisAlignment.spaceBetween,
+        children: [
+          Container(
+            width: 100,
+            height: 14,
+            color: Colors.grey.shade400,
+          ),
+          Container(
+            width: 18,
+            height: 18,
+            decoration: BoxDecoration(
+              color: Colors.grey.shade400,
+              shape: BoxShape.circle,
+            ),
+          ),
+        ],
+      ),
+    );
+  }
+
   @override
   Widget build(BuildContext context) {
     return Obx(() {
       if (controller.isLoadingServices.value) {
-        return const Center(child: CircularProgressIndicator());
+        return _skeletonSelector();
       }
 
       final serviceNames = controller.services.isEmpty

lib/model/attendance/attendence_action_button.dart

@@ -108,8 +108,10 @@ class _AttendanceActionButtonState extends State<AttendanceActionButton> {
         break;
 
       case 1:
-        final isOldCheckIn = AttendanceButtonHelper.isOlderThanDays(widget.employee.checkIn, 2);
-        final isOldCheckOut = AttendanceButtonHelper.isOlderThanDays(widget.employee.checkOut, 2);
+        final isOldCheckIn =
+            AttendanceButtonHelper.isOlderThanDays(widget.employee.checkIn, 2);
+        final isOldCheckOut =
+            AttendanceButtonHelper.isOlderThanDays(widget.employee.checkOut, 2);
 
         if (widget.employee.checkOut == null && isOldCheckIn) {
           action = 2;
@@ -167,7 +169,9 @@ class _AttendanceActionButtonState extends State<AttendanceActionButton> {
     String? markTime;
     if (actionText == ButtonActions.requestRegularize) {
       selectedTime ??= await _pickRegularizationTime(widget.employee.checkIn!);
-      markTime = selectedTime != null ? DateFormat("hh:mm a").format(selectedTime) : null;
+      markTime = selectedTime != null
+          ? DateFormat("hh:mm a").format(selectedTime)
+          : null;
     } else if (selectedTime != null) {
       markTime = DateFormat("hh:mm a").format(selectedTime);
     }
@@ -205,13 +209,17 @@ class _AttendanceActionButtonState extends State<AttendanceActionButton> {
   Widget build(BuildContext context) {
     return Obx(() {
       final controller = widget.attendanceController;
-      final isUploading = controller.uploadingStates[uniqueLogKey]?.value ?? false;
+      final isUploading =
+          controller.uploadingStates[uniqueLogKey]?.value ?? false;
       final emp = widget.employee;
 
-      final isYesterday = AttendanceButtonHelper.isLogFromYesterday(emp.checkIn, emp.checkOut);
-      final isTodayApproved = AttendanceButtonHelper.isTodayApproved(emp.activity, emp.checkIn);
+      final isYesterday =
+          AttendanceButtonHelper.isLogFromYesterday(emp.checkIn, emp.checkOut);
+      final isTodayApproved =
+          AttendanceButtonHelper.isTodayApproved(emp.activity, emp.checkIn);
       final isApprovedButNotToday =
-          AttendanceButtonHelper.isApprovedButNotToday(emp.activity, isTodayApproved);
+          AttendanceButtonHelper.isApprovedButNotToday(
+              emp.activity, isTodayApproved);
 
       final isButtonDisabled = AttendanceButtonHelper.isButtonDisabled(
         isUploading: isUploading,
@@ -272,12 +280,12 @@ class AttendanceActionButtonUI extends StatelessWidget {
           textStyle: const TextStyle(fontSize: 12),
         ),
         child: isUploading
-            ? const SizedBox(
-                width: 16,
-                height: 16,
-                child: CircularProgressIndicator(
-                  strokeWidth: 2,
-                  valueColor: AlwaysStoppedAnimation<Color>(Colors.white),
+            ? Container(
+                width: 60,
+                height: 14,
+                decoration: BoxDecoration(
+                  color: Colors.white.withOpacity(0.5),
+                  borderRadius: BorderRadius.circular(4),
                 ),
               )
             : Row(
@@ -288,7 +296,8 @@ class AttendanceActionButtonUI extends StatelessWidget {
                   if (buttonText.toLowerCase() == 'rejected')
                     const Icon(Icons.close, size: 16, color: Colors.red),
                   if (buttonText.toLowerCase() == 'requested')
-                    const Icon(Icons.hourglass_top, size: 16, color: Colors.orange),
+                    const Icon(Icons.hourglass_top,
+                        size: 16, color: Colors.orange),
                   if (['approved', 'rejected', 'requested']
                       .contains(buttonText.toLowerCase()))
                     const SizedBox(width: 4),
@@ -342,7 +351,8 @@ Future<String?> _showCommentBottomSheet(
           }
 
           return Padding(
-            padding: EdgeInsets.only(bottom: MediaQuery.of(context).viewInsets.bottom),
+            padding: EdgeInsets.only(
+                bottom: MediaQuery.of(context).viewInsets.bottom),
             child: BaseBottomSheet(
               title: sheetTitle, // 👈 now showing full sentence as title
               onCancel: () => Navigator.of(context).pop(),
@@ -375,6 +385,5 @@ Future<String?> _showCommentBottomSheet(
   );
 }
 
-
 String capitalizeFirstLetter(String text) =>
     text.isEmpty ? text : text[0].toUpperCase() + text.substring(1);

lib/model/attendance/attendence_filter_sheet.dart

@@ -39,8 +39,7 @@ class _AttendanceFilterBottomSheetState
     final endDate = widget.controller.endDateAttendance;
 
     if (startDate != null && endDate != null) {
-      final start =
-          DateTimeUtils.formatDate(startDate, 'dd MMM yyyy'); 
+      final start = DateTimeUtils.formatDate(startDate, 'dd MMM yyyy');
       final end = DateTimeUtils.formatDate(endDate, 'dd MMM yyyy');
       return "$start - $end";
     }
@@ -161,7 +160,32 @@ class _AttendanceFilterBottomSheetState
       ),
       Obx(() {
         if (widget.controller.isLoadingOrganizations.value) {
-          return const Center(child: CircularProgressIndicator());
+          return Container(
+            height: 40,
+            padding: const EdgeInsets.symmetric(horizontal: 12),
+            decoration: BoxDecoration(
+              color: Colors.grey.shade300,
+              borderRadius: BorderRadius.circular(12),
+            ),
+            child: Row(
+              mainAxisAlignment: MainAxisAlignment.spaceBetween,
+              children: [
+                Container(
+                  width: 100,
+                  height: 14,
+                  color: Colors.grey.shade400,
+                ),
+                Container(
+                  width: 18,
+                  height: 18,
+                  decoration: BoxDecoration(
+                    color: Colors.grey.shade400,
+                    shape: BoxShape.circle,
+                  ),
+                ),
+              ],
+            ),
+          );
         } else if (widget.controller.organizations.isEmpty) {
           return Center(
             child: Padding(

lib/model/attendance/regualrize_action_button.dart

@@ -3,11 +3,11 @@ import 'package:marco/helpers/utils/attendance_actions.dart';
 import 'package:marco/helpers/widgets/my_snackbar.dart';
 import 'package:marco/controller/project_controller.dart';
 import 'package:get/get.dart';
+
 enum ButtonActions { approve, reject }
 
 class RegularizeActionButton extends StatefulWidget {
-  final dynamic
-      attendanceController; 
+  final dynamic attendanceController;
   final dynamic log;
   final String uniqueLogKey;
   final ButtonActions action;
@@ -70,9 +70,11 @@ class _RegularizeActionButtonState extends State<RegularizeActionButton> {
       isUploading = true;
     });
 
-  widget.attendanceController.uploadingStates[widget.uniqueLogKey]?.value = true;
+    widget.attendanceController.uploadingStates[widget.uniqueLogKey]?.value =
+        true;
 
-  final success = await widget.attendanceController.captureAndUploadAttendance(
+    final success =
+        await widget.attendanceController.captureAndUploadAttendance(
       widget.log.id,
       widget.log.employeeId,
       selectedProjectId,
@@ -92,17 +94,18 @@ class _RegularizeActionButtonState extends State<RegularizeActionButton> {
     if (success) {
       widget.attendanceController.fetchEmployeesByProject(selectedProjectId);
       widget.attendanceController.fetchAttendanceLogs(selectedProjectId);
-    await widget.attendanceController.fetchRegularizationLogs(selectedProjectId);
+      await widget.attendanceController
+          .fetchRegularizationLogs(selectedProjectId);
       await widget.attendanceController.fetchProjectData(selectedProjectId);
     }
 
-  widget.attendanceController.uploadingStates[widget.uniqueLogKey]?.value = false;
+    widget.attendanceController.uploadingStates[widget.uniqueLogKey]?.value =
+        false;
 
     setState(() {
       isUploading = false;
     });
-}
-
+  }
 
   @override
   Widget build(BuildContext context) {
@@ -116,17 +119,19 @@ class _RegularizeActionButtonState extends State<RegularizeActionButton> {
           onPressed: isUploading ? null : _handlePress,
           style: ElevatedButton.styleFrom(
             backgroundColor: backgroundColor,
-            foregroundColor:
-                Colors.white, // Ensures visibility on all backgrounds
+            foregroundColor: Colors.white,
             padding: const EdgeInsets.symmetric(vertical: 4, horizontal: 6),
             minimumSize: const Size(60, 20),
             textStyle: const TextStyle(fontSize: 12),
           ),
           child: isUploading
-              ? const SizedBox(
-                  width: 16,
-                  height: 16,
-                  child: CircularProgressIndicator(strokeWidth: 2),
+              ? Container(
+                  width: 60,
+                  height: 14,
+                  decoration: BoxDecoration(
+                    color: Colors.white.withOpacity(0.5),
+                    borderRadius: BorderRadius.circular(4),
+                  ),
                 )
               : FittedBox(
                   fit: BoxFit.scaleDown,

lib/model/dailyTaskPlanning/assign_task_bottom_sheet .dart

@@ -153,7 +153,36 @@ class _AssignTaskBottomSheetState extends State<AssignTaskBottomSheet> {
   Widget _buildEmployeeList() {
     return Obx(() {
       if (controller.isLoading.value) {
-        return const Center(child: CircularProgressIndicator());
+        // Skeleton loader instead of CircularProgressIndicator
+        return ListView.separated(
+          shrinkWrap: true,
+          itemCount: 5, // show 5 skeleton rows
+          separatorBuilder: (_, __) => const SizedBox(height: 4),
+          itemBuilder: (context, index) {
+            return Padding(
+              padding: const EdgeInsets.symmetric(vertical: 2, horizontal: 8),
+              child: Row(
+                children: [
+                  Container(
+                    width: 24,
+                    height: 24,
+                    decoration: BoxDecoration(
+                      color: Colors.grey.shade300,
+                      borderRadius: BorderRadius.circular(4),
+                    ),
+                  ),
+                  const SizedBox(width: 8),
+                  Expanded(
+                    child: Container(
+                      height: 14,
+                      color: Colors.grey.shade300,
+                    ),
+                  ),
+                ],
+              ),
+            );
+          },
+        );
       }
 
       final selectedRoleId = controller.selectedRoleId.value;
@@ -276,8 +305,9 @@ class _AssignTaskBottomSheetState extends State<AssignTaskBottomSheet> {
             hintText: '',
             border: OutlineInputBorder(),
           ),
-          validator: (value) =>
-              this.controller.formFieldValidator(value, fieldType: validatorType),
+          validator: (value) => this
+              .controller
+              .formFieldValidator(value, fieldType: validatorType),
         ),
       ],
     );

lib/model/dailyTaskPlanning/daily_task_model.dart

@@ -16,38 +16,36 @@ class TaskModel {
     required this.assignmentDate,
     this.reportedDate,
     required this.id,
-    required this.workItem,
+    this.workItem,
     required this.workItemId,
     required this.plannedTask,
     required this.completedTask,
     required this.assignedBy,
     this.approvedBy,
-    required this.teamMembers,
-    required this.comments,
-    required this.reportedPreSignedUrls,
+    this.teamMembers = const [],
+    this.comments = const [],
+    this.reportedPreSignedUrls = const [],
   });
 
   factory TaskModel.fromJson(Map<String, dynamic> json) {
     return TaskModel(
-      assignmentDate: DateTime.parse(json['assignmentDate']),
-      reportedDate: json['reportedDate'] != null
-          ? DateTime.tryParse(json['reportedDate'])
-          : null,
-      id: json['id'],
-      workItem:
-          json['workItem'] != null ? WorkItem.fromJson(json['workItem']) : null,
-      workItemId: json['workItemId'],
-      plannedTask: (json['plannedTask'] as num).toDouble(),
-      completedTask: (json['completedTask'] as num).toDouble(),
-      assignedBy: AssignedBy.fromJson(json['assignedBy']),
-      approvedBy: json['approvedBy'] != null
-          ? AssignedBy.fromJson(json['approvedBy'])
-          : null,
-      teamMembers: (json['teamMembers'] as List)
-          .map((e) => TeamMember.fromJson(e))
-          .toList(),
-      comments:
-          (json['comments'] as List).map((e) => Comment.fromJson(e)).toList(),
+      assignmentDate: DateTime.parse(json['assignmentDate'] ?? DateTime.now().toIso8601String()),
+      reportedDate: json['reportedDate'] != null ? DateTime.tryParse(json['reportedDate']) : null,
+      id: json['id']?.toString() ?? '',
+      workItem: json['workItem'] != null ? WorkItem.fromJson(json['workItem']) : null,
+      workItemId: json['workItemId']?.toString() ?? '',
+      plannedTask: (json['plannedTask'] as num?)?.toDouble() ?? 0,
+      completedTask: (json['completedTask'] as num?)?.toDouble() ?? 0,
+      assignedBy: AssignedBy.fromJson(json['assignedBy'] ?? {}),
+      approvedBy: json['approvedBy'] != null ? AssignedBy.fromJson(json['approvedBy']) : null,
+      teamMembers: (json['teamMembers'] as List<dynamic>?)
+              ?.map((e) => TeamMember.fromJson(e))
+              .toList() ??
+          [],
+      comments: (json['comments'] as List<dynamic>?)
+              ?.map((e) => Comment.fromJson(e))
+              .toList() ??
+          [],
       reportedPreSignedUrls: (json['reportedPreSignedUrls'] as List<dynamic>?)
               ?.map((e) => e.toString())
               .toList() ??
@@ -79,8 +77,7 @@ class WorkItem {
       activityMaster: json['activityMaster'] != null
           ? ActivityMaster.fromJson(json['activityMaster'])
           : null,
-      workArea:
-          json['workArea'] != null ? WorkArea.fromJson(json['workArea']) : null,
+      workArea: json['workArea'] != null ? WorkArea.fromJson(json['workArea']) : null,
       plannedWork: (json['plannedWork'] as num?)?.toDouble(),
       completedWork: (json['completedWork'] as num?)?.toDouble(),
       preSignedUrls: (json['preSignedUrls'] as List<dynamic>?)
@@ -92,7 +89,7 @@ class WorkItem {
 }
 
 class ActivityMaster {
-  final String? id; // ✅ Added
+  final String? id;
   final String activityName;
 
   ActivityMaster({
@@ -103,13 +100,13 @@ class ActivityMaster {
   factory ActivityMaster.fromJson(Map<String, dynamic> json) {
     return ActivityMaster(
       id: json['id']?.toString(),
-      activityName: json['activityName'] ?? '',
+      activityName: json['activityName']?.toString() ?? '',
     );
   }
 }
 
 class WorkArea {
-  final String? id; // ✅ Added
+  final String? id;
   final String areaName;
   final Floor? floor;
 
@@ -122,7 +119,7 @@ class WorkArea {
   factory WorkArea.fromJson(Map<String, dynamic> json) {
     return WorkArea(
       id: json['id']?.toString(),
-      areaName: json['areaName'] ?? '',
+      areaName: json['areaName']?.toString() ?? '',
       floor: json['floor'] != null ? Floor.fromJson(json['floor']) : null,
     );
   }
@@ -136,9 +133,8 @@ class Floor {
 
   factory Floor.fromJson(Map<String, dynamic> json) {
     return Floor(
-      floorName: json['floorName'] ?? '',
-      building:
-          json['building'] != null ? Building.fromJson(json['building']) : null,
+      floorName: json['floorName']?.toString() ?? '',
+      building: json['building'] != null ? Building.fromJson(json['building']) : null,
     );
   }
 }
@@ -149,7 +145,7 @@ class Building {
   Building({required this.name});
 
   factory Building.fromJson(Map<String, dynamic> json) {
-    return Building(name: json['name'] ?? '');
+    return Building(name: json['name']?.toString() ?? '');
   }
 }
 
@@ -167,8 +163,8 @@ class AssignedBy {
   factory AssignedBy.fromJson(Map<String, dynamic> json) {
     return AssignedBy(
       id: json['id']?.toString() ?? '',
-      firstName: json['firstName'] ?? '',
-      lastName: json['lastName'],
+      firstName: json['firstName']?.toString() ?? '',
+      lastName: json['lastName']?.toString(),
     );
   }
 }
@@ -203,7 +199,7 @@ class Comment {
     required this.comment,
     required this.commentedBy,
     required this.timestamp,
-    required this.preSignedUrls,
+    this.preSignedUrls = const [],
   });
 
   factory Comment.fromJson(Map<String, dynamic> json) {
@@ -212,7 +208,9 @@ class Comment {
       commentedBy: json['employee'] != null
           ? TeamMember.fromJson(json['employee'])
           : TeamMember(id: '', firstName: '', lastName: null),
-      timestamp: DateTime.parse(json['commentDate'] ?? ''),
+      timestamp: json['commentDate'] != null
+          ? DateTime.parse(json['commentDate'])
+          : DateTime.now(),
       preSignedUrls: (json['preSignedUrls'] as List<dynamic>?)
               ?.map((e) => e.toString())
               .toList() ??

lib/model/dailyTaskPlanning/report_action_bottom_sheet.dart

@@ -147,8 +147,9 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
               floatingLabelBehavior: FloatingLabelBehavior.never,
             ),
           ),
-
           MySpacing.height(10),
+
+          // Reported Images Section
           if ((widget.taskData['reportedPreSignedUrls'] as List<dynamic>?)
                   ?.isNotEmpty ==
               true)
@@ -157,23 +158,22 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
                   widget.taskData['reportedPreSignedUrls'] ?? []),
               context: context,
             ),
-
           MySpacing.height(10),
+
+          // Report Actions Dropdown
           MyText.titleSmall("Report Actions", fontWeight: 600),
           MySpacing.height(10),
-
           Obx(() {
             if (controller.isLoadingWorkStatus.value)
               return const CircularProgressIndicator();
             return PopupMenuButton<String>(
-              onSelected: (String value) {
+              onSelected: (value) {
                 controller.selectedWorkStatusName.value = value;
                 controller.showAddTaskCheckbox.value = true;
               },
               shape: RoundedRectangleBorder(
                   borderRadius: BorderRadius.circular(12)),
-              itemBuilder: (BuildContext context) {
-                return controller.workStatus.map((status) {
+              itemBuilder: (context) => controller.workStatus.map((status) {
                 return PopupMenuItem<String>(
                   value: status.name,
                   child: Row(
@@ -188,8 +188,7 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
                     ],
                   ),
                 );
-                }).toList();
-              },
+              }).toList(),
               child: Container(
                 padding: MySpacing.xy(16, 12),
                 decoration: BoxDecoration(
@@ -211,9 +210,9 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
               ),
             );
           }),
-
           MySpacing.height(10),
 
+          // Add New Task Checkbox
           Obx(() {
             if (!controller.showAddTaskCheckbox.value)
               return const SizedBox.shrink();
@@ -221,18 +220,14 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
               data: Theme.of(context).copyWith(
                 checkboxTheme: CheckboxThemeData(
                   shape: RoundedRectangleBorder(
-                    borderRadius: BorderRadius.circular(4),
-                  ),
-                  side: const BorderSide(
-                      color: Colors.black, width: 2), 
+                      borderRadius: BorderRadius.circular(4)),
+                  side: const BorderSide(color: Colors.black, width: 2),
                   fillColor: MaterialStateProperty.resolveWith<Color>((states) {
-                    if (states.contains(MaterialState.selected)) {
+                    if (states.contains(MaterialState.selected))
                       return Colors.blueAccent;
-                    }
                     return Colors.white;
                   }),
-                  checkColor:
-                      MaterialStateProperty.all(Colors.white), 
+                  checkColor: MaterialStateProperty.all(Colors.white),
                 ),
               ),
               child: CheckboxListTile(
@@ -245,10 +240,9 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
               ),
             );
           }),
-
           MySpacing.height(24),
 
-          // ✏️ Comment Field
+          // 💬 Comment Field
           Row(
             children: [
               Icon(Icons.comment_outlined, size: 18, color: Colors.grey[700]),
@@ -258,8 +252,8 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
           ),
           MySpacing.height(8),
           TextFormField(
-            validator: controller.basicValidator.getValidation('comment'),
             controller: controller.basicValidator.getController('comment'),
+            validator: controller.basicValidator.getValidation('comment'),
             keyboardType: TextInputType.text,
             decoration: InputDecoration(
               hintText: "eg: Work done successfully",
@@ -269,10 +263,9 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
               floatingLabelBehavior: FloatingLabelBehavior.never,
             ),
           ),
-
           MySpacing.height(16),
 
-          // 📸 Image Attachments
+          // 📸 Attach Photos
           Row(
             crossAxisAlignment: CrossAxisAlignment.start,
             children: [
@@ -297,21 +290,18 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
               onCameraTap: () => controller.pickImages(fromCamera: true),
               onUploadTap: () => controller.pickImages(fromCamera: false),
               onRemoveImage: (index) => controller.removeImageAt(index),
-              onPreviewImage: (index) {
-                showDialog(
+              onPreviewImage: (index) => showDialog(
                 context: context,
                 builder: (_) => ImageViewerDialog(
                   imageSources: images,
                   initialIndex: index,
                 ),
-                );
-              },
+              ),
             );
           }),
-
           MySpacing.height(12),
 
-          // ✅ Submit/Cancel Buttons moved here
+          // ✅ Submit/Cancel Buttons
           Row(
             children: [
               Expanded(
@@ -347,7 +337,6 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
                                     ?.text
                                     .trim() ??
                                 '';
-
                             final shouldShowAddTaskSheet =
                                 controller.isAddTaskChecked.value;
 
@@ -408,10 +397,9 @@ class _ReportActionBottomSheetState extends State<ReportActionBottomSheet>
               ),
             ],
           ),
-
           MySpacing.height(12),
 
-          // 💬 Previous Comments List (only below submit)
+          // 💬 Previous Comments
           if ((widget.taskData['taskComments'] as List<dynamic>?)?.isNotEmpty ==
               true) ...[
             Row(

lib/model/employees/add_employee_bottom_sheet.dart

@@ -50,7 +50,6 @@ class _AddEmployeeBottomSheetState extends State<AddEmployeeBottomSheet>
       _controller.editingEmployeeData = widget.employeeData;
       _controller.prefillFields();
 
-      // Prepopulate hasApplicationAccess and email
       _hasApplicationAccess =
           widget.employeeData?['hasApplicationAccess'] ?? false;
 
@@ -60,22 +59,32 @@ class _AddEmployeeBottomSheetState extends State<AddEmployeeBottomSheet>
             email.toString();
       }
 
-      // Trigger UI rebuild to reflect email & checkbox
-      setState(() {});
+      final orgId = widget.employeeData?['organization_id'];
+      if (orgId != null) {
+        final org = _organizationController.organizations
+            .firstWhereOrNull((o) => o.id == orgId);
+        if (org != null) {
+          WidgetsBinding.instance.addPostFrameCallback((_) {
+            _organizationController.selectOrganization(org);
+            _controller.selectedOrganizationId = org.id;
+            _orgFieldController.text = org.name;
+          });
+        }
+      }
 
-      // Joining date
+      // ✅ Prefill Joining date
       if (_controller.joiningDate != null) {
         _joiningDateController.text =
             DateFormat('dd MMM yyyy').format(_controller.joiningDate!);
       }
 
-      // Gender
+      // ✅ Prefill Gender
       if (_controller.selectedGender != null) {
         _genderController.text =
             _controller.selectedGender!.name.capitalizeFirst ?? '';
       }
 
-      // Role
+      // ✅ Prefill Role
       _controller.fetchRoles().then((_) {
         if (_controller.selectedRoleId != null) {
           final roleName = _controller.roles.firstWhereOrNull(
@@ -91,6 +100,7 @@ class _AddEmployeeBottomSheetState extends State<AddEmployeeBottomSheet>
       _controller.fetchRoles();
     }
   }
+
   @override
   void dispose() {
     _orgFieldController.dispose();
@@ -337,8 +347,7 @@ class _AddEmployeeBottomSheetState extends State<AddEmployeeBottomSheet>
             return null;
           },
           keyboardType: TextInputType.emailAddress,
-          decoration: _inputDecoration('e.g., john.doe@example.com').copyWith(
-          ),
+          decoration: _inputDecoration('e.g., john.doe@example.com').copyWith(),
         ),
       ],
     );

lib/routes.dart

@@ -45,7 +45,7 @@ getPageRoute() {
         page: () => DashboardScreen(),
         middlewares: [AuthMiddleware()]),
     GetPage(
-      name: '/home',
+      name: '/dashboard',
       page: () => DashboardScreen(), // or your actual home screen
       middlewares: [AuthMiddleware()],
     ),

lib/view/auth/email_login_form.dart

@@ -123,20 +123,24 @@ class _EmailLoginFormState extends State<EmailLoginForm> with UIMixin {
                 ),
                 MySpacing.height(28),
                 Center(
-                  child: MyButton.rounded(
-                    onPressed: controller.onLogin,
+                  child: Obx(() {
+                    final isLoading = controller.isLoading.value;
+                    return MyButton.rounded(
+                      onPressed: isLoading
+                          ? null
+                          : controller.onLogin, 
                       elevation: 2,
                       padding: MySpacing.xy(80, 16),
                       borderRadiusAll: 10,
                       backgroundColor: contentTheme.brandRed,
                       child: MyText.labelLarge(
-                      'Login',
+                        isLoading ? 'Logging in...' : 'Login',
                         fontWeight: 700,
                         color: contentTheme.onPrimary,
                       ),
+                    );
+                  }),
                 ),
-                ),
-                
               ],
             ),
           );

lib/view/auth/login_screen.dart

@@ -37,7 +37,7 @@ class _LoginScreenState extends State<LoginScreen> with UIMixin {
         builder: (_) {
           return Obx(() {
             if (controller.isLoading.value) {
-              return const Center(child: CircularProgressIndicator());
+              return const Center(child: LinearProgressIndicator());
             }
 
             return Form(

lib/view/dashboard/dashboard_screen.dart

@@ -84,8 +84,6 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
   /// Project Progress Chart Section
   Widget _buildProjectProgressChartSection() {
     return Obx(() {
-  
-
       if (dashboardController.projectChartData.isEmpty) {
         return const Padding(
           padding: EdgeInsets.all(16),
@@ -110,7 +108,6 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
   /// Attendance Chart Section
   Widget _buildAttendanceChartSection() {
     return Obx(() {
-
       final isAttendanceAllowed = menuController.isMenuAllowed("Attendance");
 
       if (!isAttendanceAllowed) {
@@ -212,7 +209,7 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
         return _buildLoadingSkeleton(context);
       }
 
-      if (menuController.hasError.value && menuController.menuItems.isEmpty) {
+      if (menuController.hasError.value || menuController.menuItems.isEmpty) {
         return Padding(
           padding: const EdgeInsets.all(16),
           child: Center(
@@ -224,6 +221,10 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
         );
       }
 
+      final projectController = Get.find<ProjectController>();
+      final isProjectSelected = projectController.selectedProject != null;
+
+      // Keep previous stat items (icons, title, routes)
       final stats = [
         _StatItem(LucideIcons.scan_face, "Attendance", contentTheme.success,
             DashboardScreen.attendanceRoute),
@@ -241,8 +242,16 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
             DashboardScreen.documentMainPageRoute),
       ];
 
-      final projectController = Get.find<ProjectController>();
-      final isProjectSelected = projectController.selectedProject != null;
+      // Safe menu check function to avoid exceptions
+      bool _isMenuAllowed(String menuTitle) {
+        try {
+          return menuController.menuItems.isNotEmpty
+              ? menuController.isMenuAllowed(menuTitle)
+              : false;
+        } catch (e) {
+          return false;
+        }
+      }
 
       return Column(
         crossAxisAlignment: CrossAxisAlignment.start,
@@ -250,7 +259,6 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
           if (!isProjectSelected) _buildNoProjectMessage(),
           LayoutBuilder(
             builder: (context, constraints) {
-              // ✅ smaller width cards → fit more in a row
               int crossAxisCount = (constraints.maxWidth ~/ 80).clamp(2, 8);
               double cardWidth =
                   (constraints.maxWidth - (crossAxisCount - 1) * 6) /
@@ -261,14 +269,10 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
                 runSpacing: 6,
                 alignment: WrapAlignment.start,
                 children: stats
-                    .where((stat) {
-                      if (stat.title == "Documents") return true;
-                      return menuController.isMenuAllowed(stat.title);
-                    })
+                    .where((stat) => _isMenuAllowed(stat.title))
                     .map((stat) =>
                         _buildStatCard(stat, isProjectSelected, cardWidth))
-                    .toList()
-                    .cast<Widget>(),
+                    .toList(),
               );
             },
           ),

lib/view/directory/contact_detail_screen.dart

@@ -360,15 +360,13 @@ class _ContactDetailScreenState extends State<ContactDetailScreen> {
           [...activeComments, ...inactiveComments].reversed.toList();
       final editingId = directoryController.editingCommentId.value;
 
-      if (comments.isEmpty) {
-        return Center(
-          child: MyText.bodyLarge("No notes yet.", color: Colors.grey),
-        );
-      }
-
       return Stack(
         children: [
-          MyRefreshIndicator(
+          comments.isEmpty
+              ? Center(
+                  child: MyText.bodyLarge("No notes yet.", color: Colors.grey),
+                )
+              : MyRefreshIndicator(
                   onRefresh: () async {
                     await directoryController.fetchCommentsForContact(contactId,
                         active: true);
@@ -382,8 +380,8 @@ class _ContactDetailScreenState extends State<ContactDetailScreen> {
                       padding: const EdgeInsets.only(bottom: 100),
                       itemCount: comments.length,
                       separatorBuilder: (_, __) => MySpacing.height(14),
-                itemBuilder: (_, index) =>
-                    _buildCommentItem(comments[index], editingId, contactId),
+                      itemBuilder: (_, index) => _buildCommentItem(
+                          comments[index], editingId, contactId),
                     ),
                   ),
                 ),

lib/view/document/document_details_page.dart

@@ -27,8 +27,8 @@ class _DocumentDetailsPageState extends State<DocumentDetailsPage> {
   final DocumentDetailsController controller =
       Get.find<DocumentDetailsController>();
 
-  final PermissionController permissionController =
-      Get.find<PermissionController>();
+  final permissionController = Get.put(PermissionController());
+
   @override
   void initState() {
     super.initState();

lib/view/document/user_document_screen.dart

@@ -1,24 +1,24 @@
+import 'dart:convert';
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
-import 'package:marco/controller/document/user_document_controller.dart';
-import 'package:marco/controller/project_controller.dart';
-import 'package:marco/helpers/widgets/my_spacing.dart';
-import 'package:marco/helpers/widgets/my_text.dart';
-import 'package:marco/helpers/widgets/my_refresh_indicator.dart';
-import 'package:marco/helpers/utils/permission_constants.dart';
-import 'package:marco/model/document/user_document_filter_bottom_sheet.dart';
-import 'package:marco/model/document/documents_list_model.dart';
 import 'package:intl/intl.dart';
-import 'package:marco/helpers/widgets/my_custom_skeleton.dart';
-import 'package:marco/model/document/document_upload_bottom_sheet.dart';
+import 'package:marco/controller/document/document_details_controller.dart';
 import 'package:marco/controller/document/document_upload_controller.dart';
-import 'package:marco/view/document/document_details_page.dart';
+import 'package:marco/controller/document/user_document_controller.dart';
+import 'package:marco/controller/permission_controller.dart';
+import 'package:marco/controller/project_controller.dart';
+import 'package:marco/helpers/utils/permission_constants.dart';
 import 'package:marco/helpers/widgets/custom_app_bar.dart';
 import 'package:marco/helpers/widgets/my_confirmation_dialog.dart';
+import 'package:marco/helpers/widgets/my_refresh_indicator.dart';
+import 'package:marco/helpers/widgets/my_spacing.dart';
+import 'package:marco/helpers/widgets/my_text.dart';
 import 'package:marco/helpers/widgets/my_snackbar.dart';
-import 'package:marco/controller/permission_controller.dart';
-import 'package:marco/controller/document/document_details_controller.dart';
-import 'dart:convert';
+import 'package:marco/helpers/widgets/my_custom_skeleton.dart';
+import 'package:marco/model/document/document_upload_bottom_sheet.dart';
+import 'package:marco/model/document/documents_list_model.dart';
+import 'package:marco/model/document/user_document_filter_bottom_sheet.dart';
+import 'package:marco/view/document/document_details_page.dart';
 
 class UserDocumentsPage extends StatefulWidget {
   final String? entityId;
@@ -36,10 +36,9 @@ class UserDocumentsPage extends StatefulWidget {
 
 class _UserDocumentsPageState extends State<UserDocumentsPage> {
   final DocumentController docController = Get.put(DocumentController());
-  final PermissionController permissionController =
-      Get.find<PermissionController>();
-  final DocumentDetailsController controller =
-      Get.put(DocumentDetailsController());
+  final PermissionController permissionController = Get.put(PermissionController());
+  final DocumentDetailsController controller = Get.put(DocumentDetailsController());
+
   String get entityTypeId => widget.isEmployee
       ? Permissions.employeeEntity
       : Permissions.projectEntity;
@@ -68,12 +67,9 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
   }
 
   Widget _buildDocumentTile(DocumentItem doc, bool showDateHeader) {
-    final uploadDate =
-        DateFormat("dd MMM yyyy").format(doc.uploadedAt.toLocal());
-
+    final uploadDate = DateFormat("dd MMM yyyy").format(doc.uploadedAt.toLocal());
     final uploader = doc.uploadedBy.firstName.isNotEmpty
-        ? "Added by ${doc.uploadedBy.firstName} ${doc.uploadedBy.lastName}"
-            .trim()
+        ? "Added by ${doc.uploadedBy.firstName} ${doc.uploadedBy.lastName}".trim()
         : "Added by you";
 
     return Column(
@@ -91,7 +87,6 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
           ),
         InkWell(
           onTap: () {
-            // 👉 Navigate to details page
             Get.to(() => DocumentDetailsPage(documentId: doc.id));
           },
           child: Container(
@@ -146,11 +141,12 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                     ],
                   ),
                 ),
-                PopupMenuButton<String>(
+                Obx(() {
+                  // React to permission changes
+                  return PopupMenuButton<String>(
                     icon: const Icon(Icons.more_vert, color: Colors.black54),
                     onSelected: (value) async {
                       if (value == "delete") {
-                      // existing delete flow (unchanged)
                         final result = await showDialog<bool>(
                           context: context,
                           builder: (_) => ConfirmDialog(
@@ -182,8 +178,7 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                                   message: "Failed to delete document",
                                   type: SnackbarType.error,
                                 );
-                              throw Exception(
-                                  "Failed to delete"); // keep dialog open
+                                throw Exception("Failed to delete");
                               }
                             },
                           ),
@@ -192,7 +187,6 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                           debugPrint("✅ Document deleted and removed from list");
                         }
                       } else if (value == "restore") {
-                      // existing activate flow (unchanged)
                         final success = await docController.toggleDocumentActive(
                           doc.id,
                           isActive: true,
@@ -203,7 +197,7 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                         if (success) {
                           showAppSnackbar(
                             title: "Restored",
-                          message: "Document reastored successfully",
+                            message: "Document restored successfully",
                             type: SnackbarType.success,
                           );
                         } else {
@@ -217,21 +211,20 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                     },
                     itemBuilder: (context) => [
                       if (doc.isActive &&
-                        permissionController
-                            .hasPermission(Permissions.deleteDocument))
+                          permissionController.hasPermission(Permissions.deleteDocument))
                         const PopupMenuItem(
                           value: "delete",
                           child: Text("Delete"),
                         )
                       else if (!doc.isActive &&
-                        permissionController
-                            .hasPermission(Permissions.modifyDocument))
+                          permissionController.hasPermission(Permissions.modifyDocument))
                         const PopupMenuItem(
                           value: "restore",
                           child: Text("Restore"),
                         ),
                     ],
-                ),
+                  );
+                }),
               ],
             ),
           ),
@@ -267,7 +260,6 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
       padding: MySpacing.xy(8, 8),
       child: Row(
         children: [
-          // 🔍 Search Bar
           Expanded(
             child: SizedBox(
               height: 35,
@@ -283,15 +275,13 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                 },
                 decoration: InputDecoration(
                   contentPadding: const EdgeInsets.symmetric(horizontal: 12),
-                  prefixIcon:
-                      const Icon(Icons.search, size: 20, color: Colors.grey),
+                  prefixIcon: const Icon(Icons.search, size: 20, color: Colors.grey),
                   suffixIcon: ValueListenableBuilder<TextEditingValue>(
                     valueListenable: docController.searchController,
                     builder: (context, value, _) {
                       if (value.text.isEmpty) return const SizedBox.shrink();
                       return IconButton(
-                        icon: const Icon(Icons.clear,
-                            size: 20, color: Colors.grey),
+                        icon: const Icon(Icons.clear, size: 20, color: Colors.grey),
                         onPressed: () {
                           docController.searchController.clear();
                           docController.searchQuery.value = '';
@@ -320,8 +310,6 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
             ),
           ),
           MySpacing.width(8),
-
-          // 🛠️ Filter Icon with indicator
           Obx(() {
             final isFilterActive = docController.hasActiveFilters();
             return Stack(
@@ -337,18 +325,13 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                   child: IconButton(
                     padding: EdgeInsets.zero,
                     constraints: BoxConstraints(),
-                    icon: Icon(
-                      Icons.tune,
-                      size: 20,
-                      color: Colors.black87,
-                    ),
+                    icon: Icon(Icons.tune, size: 20, color: Colors.black87),
                     onPressed: () {
                       showModalBottomSheet(
                         context: context,
                         isScrollControlled: true,
                         shape: const RoundedRectangleBorder(
-                          borderRadius:
-                              BorderRadius.vertical(top: Radius.circular(5)),
+                          borderRadius: BorderRadius.vertical(top: Radius.circular(5)),
                         ),
                         builder: (_) => UserDocumentFilterBottomSheet(
                           entityId: resolvedEntityId,
@@ -375,8 +358,6 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
             );
           }),
           MySpacing.width(10),
-
-          // ⋮ Menu (Show Inactive toggle)
           Container(
             height: 35,
             width: 35,
@@ -387,8 +368,7 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
             ),
             child: PopupMenuButton<int>(
               padding: EdgeInsets.zero,
-              icon:
-                  const Icon(Icons.more_vert, size: 20, color: Colors.black87),
+              icon: const Icon(Icons.more_vert, size: 20, color: Colors.black87),
               shape: RoundedRectangleBorder(
                 borderRadius: BorderRadius.circular(5),
               ),
@@ -439,8 +419,7 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
 
   Widget _buildStatusHeader() {
     return Obx(() {
-      final isInactive = docController.showInactive.value;
-      if (!isInactive) return const SizedBox.shrink(); // hide when active
+      if (!docController.showInactive.value) return const SizedBox.shrink();
 
       return Container(
         width: double.infinity,
@@ -448,18 +427,11 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
         color: Colors.red.shade50,
         child: Row(
           children: [
-            Icon(
-              Icons.visibility_off,
-              color: Colors.red,
-              size: 18,
-            ),
+            Icon(Icons.visibility_off, color: Colors.red, size: 18),
             const SizedBox(width: 8),
             Text(
               "Showing Deleted Documents",
-              style: TextStyle(
-                color: Colors.red,
-                fontWeight: FontWeight.w600,
-              ),
+              style: TextStyle(color: Colors.red, fontWeight: FontWeight.w600),
             ),
           ],
         ),
@@ -468,7 +440,11 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
   }
 
   Widget _buildBody(BuildContext context) {
-    // 🔒 Check for viewDocument permission
+    return Obx(() {
+      if (permissionController.permissions.isEmpty) {
+        return Center(child: CircularProgressIndicator());
+      }
+
       if (!permissionController.hasPermission(Permissions.viewDocument)) {
         return Center(
           child: Column(
@@ -491,7 +467,6 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
         );
       }
 
-    return Obx(() {
       if (docController.isLoading.value && docController.documents.isEmpty) {
         return SingleChildScrollView(
           physics: const NeverScrollableScrollPhysics(),
@@ -510,8 +485,7 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                 onRefresh: () async {
                   final combinedFilter = {
                     'uploadedByIds': docController.selectedUploadedBy.toList(),
-                    'documentCategoryIds':
-                        docController.selectedCategory.toList(),
+                    'documentCategoryIds': docController.selectedCategory.toList(),
                     'documentTypeIds': docController.selectedType.toList(),
                     'documentTagIds': docController.selectedTag.toList(),
                   };
@@ -525,9 +499,7 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                 },
                 child: ListView(
                   physics: const AlwaysScrollableScrollPhysics(),
-                  padding: docs.isEmpty
-                      ? null
-                      : const EdgeInsets.fromLTRB(0, 0, 0, 80),
+                  padding: docs.isEmpty ? null : const EdgeInsets.fromLTRB(0, 0, 0, 80),
                   children: docs.isEmpty
                       ? [
                           SizedBox(
@@ -543,8 +515,8 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                             final currentDate = DateFormat("dd MMM yyyy")
                                 .format(doc.uploadedAt.toLocal());
                             final prevDate = index > 0
-                                ? DateFormat("dd MMM yyyy").format(
-                                    docs[index - 1].uploadedAt.toLocal())
+                                ? DateFormat("dd MMM yyyy")
+                                    .format(docs[index - 1].uploadedAt.toLocal())
                                 : null;
 
                             final showDateHeader = currentDate != prevDate;
@@ -591,8 +563,10 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
             )
           : null,
       body: _buildBody(context),
-      floatingActionButton: permissionController
-              .hasPermission(Permissions.uploadDocument)
+      floatingActionButton: Obx(() {
+        if (permissionController.permissions.isEmpty) return SizedBox.shrink();
+
+        return permissionController.hasPermission(Permissions.uploadDocument)
             ? FloatingActionButton.extended(
                 onPressed: () {
                   final uploadController = Get.put(DocumentUploadController());
@@ -642,7 +616,8 @@ class _UserDocumentsPageState extends State<UserDocumentsPage> {
                 ),
                 backgroundColor: Colors.red,
               )
-          : null,
+            : SizedBox.shrink();
+      }),
       floatingActionButtonLocation: FloatingActionButtonLocation.endFloat,
     );
   }

lib/view/employees/employee_detail_screen.dart

@@ -30,8 +30,8 @@ class EmployeeDetailPage extends StatefulWidget {
 class _EmployeeDetailPageState extends State<EmployeeDetailPage> {
   final EmployeesScreenController controller =
       Get.put(EmployeesScreenController());
-  final PermissionController _permissionController =
-      Get.find<PermissionController>();
+  final PermissionController permissionController =
+      Get.put(PermissionController());
 
   @override
   void initState() {
@@ -272,6 +272,7 @@ class _EmployeeDetailPageState extends State<EmployeeDetailPage> {
                                 'job_role_id': employee.jobRoleId,
                                 'joining_date':
                                     employee.joiningDate?.toIso8601String(),
+                                'organization_id': employee.organizationId,
                               },
                             ),
                           );
@@ -292,7 +293,7 @@ class _EmployeeDetailPageState extends State<EmployeeDetailPage> {
         );
       }),
       floatingActionButton: Obx(() {
-        if (!_permissionController.hasPermission(Permissions.assignToProject)) {
+        if (!permissionController.hasPermission(Permissions.assignToProject)) {
           return const SizedBox.shrink();
         }
         if (controller.isLoadingEmployeeDetails.value ||

lib/view/employees/employees_screen.dart

@@ -29,8 +29,8 @@ class EmployeesScreen extends StatefulWidget {
 class _EmployeesScreenState extends State<EmployeesScreen> with UIMixin {
   final EmployeesScreenController _employeeController =
       Get.put(EmployeesScreenController());
-  final PermissionController _permissionController =
-      Get.find<PermissionController>();
+  final PermissionController permissionController =
+      Get.put(PermissionController());
   final TextEditingController _searchController = TextEditingController();
   final RxList<EmployeeModel> _filteredEmployees = <EmployeeModel>[].obs;
   final OrganizationController _organizationController =
@@ -248,7 +248,16 @@ class _EmployeesScreenState extends State<EmployeesScreen> with UIMixin {
   }
 
   Widget _buildFloatingActionButton() {
-    if (!_permissionController.hasPermission(Permissions.manageEmployees)) {
+    return Obx(() {
+      // Show nothing while permissions are loading
+      if (permissionController.isLoading.value) {
+        return const SizedBox.shrink();
+      }
+
+      // Show FAB only if user has Manage Employees permission
+      final hasPermission =
+          permissionController.hasPermission(Permissions.manageEmployees);
+      if (!hasPermission) {
         return const SizedBox.shrink();
       }
 
@@ -262,7 +271,10 @@ class _EmployeesScreenState extends State<EmployeesScreen> with UIMixin {
             borderRadius: BorderRadius.circular(28),
             boxShadow: const [
               BoxShadow(
-                color: Colors.black26, blurRadius: 6, offset: Offset(0, 3))
+                color: Colors.black26,
+                blurRadius: 6,
+                offset: Offset(0, 3),
+              )
             ],
           ),
           child: const Row(
@@ -275,6 +287,7 @@ class _EmployeesScreenState extends State<EmployeesScreen> with UIMixin {
           ),
         ),
       );
+    });
   }
 
   Widget _buildSearchAndActionRow() {
@@ -371,7 +384,9 @@ class _EmployeesScreenState extends State<EmployeesScreen> with UIMixin {
   }
 
   Widget _buildPopupMenu() {
-    if (!_permissionController.hasPermission(Permissions.viewAllEmployees)) {
+    return Obx(() {
+      if (permissionController.isLoading.value ||
+          !permissionController.hasPermission(Permissions.viewAllEmployees)) {
         return const SizedBox.shrink();
       }
 
@@ -425,6 +440,7 @@ class _EmployeesScreenState extends State<EmployeesScreen> with UIMixin {
           ),
         ],
       );
+    });
   }
 
   Widget _buildEmployeeList() {

lib/view/expense/expense_detail_screen.dart

@@ -33,7 +33,7 @@ class ExpenseDetailScreen extends StatefulWidget {
 class _ExpenseDetailScreenState extends State<ExpenseDetailScreen> {
   final controller = Get.put(ExpenseDetailController());
   final projectController = Get.find<ProjectController>();
-  final permissionController = Get.find<PermissionController>();
+final permissionController = Get.put(PermissionController());
 
   EmployeeInfo? employeeInfo;
   final RxBool canSubmit = false.obs;

lib/view/expense/expense_screen.dart

@@ -26,7 +26,7 @@ class _ExpenseMainScreenState extends State<ExpenseMainScreen>
   final searchController = TextEditingController();
   final expenseController = Get.put(ExpenseController());
   final projectController = Get.find<ProjectController>();
-  final permissionController = Get.find<PermissionController>();
+final permissionController = Get.put(PermissionController());
 
   @override
   void initState() {
@@ -81,7 +81,7 @@ class _ExpenseMainScreenState extends State<ExpenseMainScreen>
             .toList();
   }
 
-  @override
+@override
 Widget build(BuildContext context) {
   return Scaffold(
     backgroundColor: Colors.white,
@@ -106,7 +106,7 @@ Widget build(BuildContext context) {
         // ---------------- Gray background for rest ----------------
         Expanded(
           child: Container(
-            color: Colors.grey[100], // Light gray background
+            color: Colors.grey[100],
             child: Column(
               children: [
                 // ---------------- Search ----------------
@@ -137,14 +137,24 @@ Widget build(BuildContext context) {
       ],
     ),
 
-    floatingActionButton:
-        permissionController.hasPermission(Permissions.expenseUpload)
+    // ✅ FAB reacts only to upload permission
+    floatingActionButton: Obx(() {
+      // Show loader or hide FAB while permissions are loading
+      if (permissionController.permissions.isEmpty) {
+        return const SizedBox.shrink();
+      }
+
+      final canUpload =
+          permissionController.hasPermission(Permissions.expenseUpload);
+
+      return canUpload
           ? FloatingActionButton(
               backgroundColor: Colors.red,
               onPressed: showAddExpenseBottomSheet,
               child: const Icon(Icons.add, color: Colors.white),
             )
-            : null,
+          : const SizedBox.shrink();
+    }),
   );
 }
 

lib/view/layouts/left_bar.dart

@@ -82,7 +82,7 @@ class _LeftBarState extends State<LeftBar>
               child: Padding(
                 padding: EdgeInsets.only(top: 50),
                 child: InkWell(
-                  onTap: () => Get.toNamed('/home'),
+                  onTap: () => Get.toNamed('/dashboard'),
                   child: Image.asset(
                     (ThemeCustomizer.instance.theme == ThemeMode.light
                         ? (widget.isCondensed

lib/view/layouts/user_profile_right_bar.dart

@@ -9,10 +9,12 @@ import 'package:marco/helpers/widgets/my_text.dart';
 import 'package:marco/helpers/widgets/avatar.dart';
 import 'package:marco/model/employees/employee_info.dart';
 import 'package:marco/controller/auth/mpin_controller.dart';
-import 'package:marco/controller/tenant/tenant_selection_controller.dart';
 import 'package:marco/view/employees/employee_profile_screen.dart';
 import 'package:marco/helpers/services/tenant_service.dart';
 import 'package:marco/view/tenant/tenant_selection_screen.dart';
+import 'package:marco/controller/tenant/tenant_switch_controller.dart';
+
+
 
 class UserProfileBar extends StatefulWidget {
   final bool isCondensed;
@@ -27,21 +29,13 @@ class _UserProfileBarState extends State<UserProfileBar>
   late EmployeeInfo employeeInfo;
   bool _isLoading = true;
   bool hasMpin = true;
-  late final TenantSelectionController _tenantController;
 
   @override
   void initState() {
     super.initState();
-    _tenantController = Get.put(TenantSelectionController());
     _initData();
   }
 
-  @override
-  void dispose() {
-    Get.delete<TenantSelectionController>();
-    super.dispose();
-  }
-
   Future<void> _initData() async {
     employeeInfo = LocalStorage.getEmployeeInfo()!;
     hasMpin = await LocalStorage.getIsMpin();
@@ -122,13 +116,20 @@ class _UserProfileBarState extends State<UserProfileBar>
   }
 
   /// Row widget to switch tenant with popup menu (button only)
-  Widget _switchTenantRow() {
+ /// Row widget to switch tenant with popup menu (button only)
+Widget _switchTenantRow() {
+  // Use the dedicated switch controller
+  final TenantSwitchController tenantSwitchController = 
+      Get.put(TenantSwitchController());
+
   return Padding(
     padding: const EdgeInsets.symmetric(horizontal: 12, vertical: 6),
     child: Obx(() {
-        if (_tenantController.isLoading.value) return _loadingTenantContainer();
+      if (tenantSwitchController.isLoading.value) {
+        return _loadingTenantContainer();
+      }
 
-        final tenants = _tenantController.tenants;
+      final tenants = tenantSwitchController.tenants;
       if (tenants.isEmpty) return _noTenantContainer();
 
       final selectedTenant = TenantService.currentTenant;
@@ -145,7 +146,7 @@ class _UserProfileBarState extends State<UserProfileBar>
 
       return PopupMenuButton<String>(
         onSelected: (tenantId) =>
-              _tenantController.onTenantSelected(tenantId),
+            tenantSwitchController.switchTenant(tenantId),
         itemBuilder: (_) => sortedTenants.map((tenant) {
           return PopupMenuItem(
             value: tenant.id,
@@ -208,7 +209,8 @@ class _UserProfileBarState extends State<UserProfileBar>
       );
     }),
   );
-  }
+}
+
 
   Widget _loadingTenantContainer() => Container(
         padding: const EdgeInsets.symmetric(vertical: 12, horizontal: 16),

lib/view/my_app.dart

@@ -5,7 +5,6 @@ import 'package:provider/provider.dart';
 
 import 'package:marco/helpers/services/app_logger.dart';
 import 'package:marco/helpers/extensions/app_localization_delegate.dart';
-import 'package:marco/helpers/services/auth_service.dart';
 import 'package:marco/helpers/services/localizations/language.dart';
 import 'package:marco/helpers/services/navigation_services.dart';
 import 'package:marco/helpers/services/storage/local_storage.dart';
@@ -19,22 +18,21 @@ class MyApp extends StatelessWidget {
 
   Future<String> _getInitialRoute() async {
     try {
-      if (!AuthService.isLoggedIn) {
+      final token = LocalStorage.getJwtToken();
+      if (token == null || token.isEmpty) {
         logSafe("User not logged in. Routing to /auth/login-option");
         return "/auth/login-option";
       }
 
       final bool hasMpin = LocalStorage.getIsMpin();
-      logSafe("MPIN enabled: $hasMpin",  );
-
       if (hasMpin) {
         await LocalStorage.setBool("mpin_verified", false);
-        logSafe("Routing to /auth/mpin-auth and setting mpin_verified to false");
+        logSafe("Routing to /auth/mpin-auth");
         return "/auth/mpin-auth";
-      } else {
-        logSafe("MPIN not enabled. Routing to /dashboard");
-        return "/dashboard";
       }
+
+      logSafe("No MPIN. Routing to /dashboard");
+      return "/dashboard";
     } catch (e, stacktrace) {
       logSafe("Error determining initial route",
           level: LogLevel.error, error: e, stackTrace: stacktrace);

lib/view/splash_screen.dart

@@ -0,0 +1,120 @@
+import 'package:flutter/material.dart';
+import 'package:marco/images.dart';
+
+class SplashScreen extends StatefulWidget {
+  final String? message;
+  final double? logoSize;
+  final Color? backgroundColor;
+
+  const SplashScreen({
+    super.key,
+    this.message,
+    this.logoSize = 120,
+    this.backgroundColor = Colors.white,
+  });
+
+  @override
+  State<SplashScreen> createState() => _SplashScreenState();
+}
+
+class _SplashScreenState extends State<SplashScreen>
+    with SingleTickerProviderStateMixin {
+  late AnimationController _controller;
+  late Animation<double> _animation;
+
+  @override
+  void initState() {
+    super.initState();
+
+    _controller = AnimationController(
+      duration: const Duration(seconds: 1),
+      vsync: this,
+    )..repeat(reverse: true);
+
+    _animation = Tween<double>(begin: 0.0, end: 8.0).animate(
+      CurvedAnimation(parent: _controller, curve: Curves.easeInOut),
+    );
+  }
+
+  @override
+  void dispose() {
+    _controller.dispose();
+    super.dispose();
+  }
+
+  Widget _buildAnimatedDots() {
+    return Row(
+      mainAxisAlignment: MainAxisAlignment.center,
+      children: List.generate(3, (index) {
+        return AnimatedBuilder(
+          animation: _animation,
+          builder: (context, child) {
+            double opacity;
+            if (index == 0) {
+              opacity = (0.3 + _animation.value / 8).clamp(0.0, 1.0);
+            } else if (index == 1) {
+              opacity = (0.3 + (_animation.value / 8)).clamp(0.0, 1.0);
+            } else {
+              opacity = (0.3 + (1 - _animation.value / 8)).clamp(0.0, 1.0);
+            }
+            return Container(
+              margin: const EdgeInsets.symmetric(horizontal: 4),
+              width: 10,
+              height: 10,
+              decoration: BoxDecoration(
+                color: Colors.blueAccent.withOpacity(opacity),
+                shape: BoxShape.circle,
+              ),
+            );
+          },
+        );
+      }),
+    );
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    return Scaffold(
+      backgroundColor: widget.backgroundColor,
+      body: SafeArea(
+        child: Center(
+          child: Column(
+            mainAxisAlignment: MainAxisAlignment.center,
+            children: [
+              // Logo with slight bounce animation
+              ScaleTransition(
+                scale: Tween(begin: 0.8, end: 1.0).animate(
+                  CurvedAnimation(
+                    parent: _controller,
+                    curve: Curves.easeInOut,
+                  ),
+                ),
+                child: SizedBox(
+                  width: widget.logoSize,
+                  height: widget.logoSize,
+                  child: Image.asset(Images.logoDark),
+                ),
+              ),
+
+              const SizedBox(height: 20),
+              // Text message
+              if (widget.message != null)
+                Text(
+                  widget.message!,
+                  textAlign: TextAlign.center,
+                  style: const TextStyle(
+                    fontSize: 18,
+                    fontWeight: FontWeight.w600,
+                    color: Colors.black87,
+                  ),
+                ),
+              const SizedBox(height: 30),
+              // Animated loading dots
+              _buildAnimatedDots(),
+            ],
+          ),
+        ),
+      ),
+    );
+  }
+}

lib/view/taskPlanning/daily_progress.dart

@@ -41,7 +41,7 @@ class _DailyProgressReportScreenState extends State<DailyProgressReportScreen>
   final DailyTaskController dailyTaskController =
       Get.put(DailyTaskController());
   final PermissionController permissionController =
-      Get.find<PermissionController>();
+      Get.put(PermissionController());
   final ProjectController projectController = Get.find<ProjectController>();
   final ServiceController serviceController = Get.put(ServiceController());
   final ScrollController _scrollController = ScrollController();

lib/view/tenant/tenant_selection_screen.dart

@@ -6,6 +6,7 @@ import 'package:marco/helpers/utils/mixins/ui_mixin.dart';
 import 'package:marco/helpers/widgets/my_text.dart';
 import 'package:marco/images.dart';
 import 'package:marco/controller/tenant/tenant_selection_controller.dart';
+import 'package:marco/view/splash_screen.dart';
 
 class TenantSelectionScreen extends StatefulWidget {
   const TenantSelectionScreen({super.key});
@@ -20,24 +21,23 @@ class _TenantSelectionScreenState extends State<TenantSelectionScreen>
   late final AnimationController _logoAnimController;
   late final Animation<double> _logoAnimation;
   final bool _isBetaEnvironment = ApiEndpoints.baseUrl.contains("stage");
-  bool _isLoading = false;
 
   @override
   void initState() {
     super.initState();
     _controller = Get.put(TenantSelectionController());
+
     _logoAnimController = AnimationController(
       vsync: this,
       duration: const Duration(milliseconds: 800),
     );
+
     _logoAnimation = CurvedAnimation(
       parent: _logoAnimController,
       curve: Curves.easeOutBack,
     );
-    _logoAnimController.forward();
 
-    // 🔥 Tell controller this is tenant selection screen
-    _controller.loadTenants(fromTenantSelectionScreen: true);
+    _logoAnimController.forward();
   }
 
   @override
@@ -48,13 +48,17 @@ class _TenantSelectionScreenState extends State<TenantSelectionScreen>
   }
 
   Future<void> _onTenantSelected(String tenantId) async {
-    setState(() => _isLoading = true);
     await _controller.onTenantSelected(tenantId);
-    setState(() => _isLoading = false);
   }
 
   @override
   Widget build(BuildContext context) {
+    return Obx(() {
+      // Splash screen for auto-selection
+      if (_controller.isAutoSelecting.value) {
+        return const SplashScreen();
+      }
+
       return Scaffold(
         body: Stack(
           children: [
@@ -81,10 +85,9 @@ class _TenantSelectionScreenState extends State<TenantSelectionScreen>
                                   const _BetaBadge(),
                                 ],
                                 const SizedBox(height: 36),
-                              // Tenant list directly reacts to controller
                                 TenantCardList(
                                   controller: _controller,
-                                isLoading: _isLoading,
+                                  isLoading: _controller.isLoading.value,
                                   onTenantSelected: _onTenantSelected,
                                 ),
                               ],
@@ -100,9 +103,11 @@ class _TenantSelectionScreenState extends State<TenantSelectionScreen>
           ],
         ),
       );
+    });
   }
 }
 
+/// Animated Logo Widget
 class _AnimatedLogo extends StatelessWidget {
   final Animation<double> animation;
   const _AnimatedLogo({required this.animation});
@@ -132,6 +137,7 @@ class _AnimatedLogo extends StatelessWidget {
   }
 }
 
+/// Welcome Texts
 class _WelcomeTexts extends StatelessWidget {
   const _WelcomeTexts();
 
@@ -148,7 +154,7 @@ class _WelcomeTexts extends StatelessWidget {
         ),
         const SizedBox(height: 10),
         MyText(
-          "Please select which dashboard you want to explore!.",
+          "Please select which dashboard you want to explore!",
           fontSize: 14,
           color: Colors.black54,
           textAlign: TextAlign.center,
@@ -158,6 +164,7 @@ class _WelcomeTexts extends StatelessWidget {
   }
 }
 
+/// Beta Badge
 class _BetaBadge extends StatelessWidget {
   const _BetaBadge();
 
@@ -179,6 +186,7 @@ class _BetaBadge extends StatelessWidget {
   }
 }
 
+/// Tenant Card List
 class TenantCardList extends StatelessWidget {
   final TenantSelectionController controller;
   final bool isLoading;
@@ -194,10 +202,9 @@ class TenantCardList extends StatelessWidget {
   Widget build(BuildContext context) {
     return Obx(() {
       if (controller.isLoading.value || isLoading) {
-        return const Center(
-          child: CircularProgressIndicator(strokeWidth: 2),
-        );
+        return const Center(child: CircularProgressIndicator(strokeWidth: 2));
       }
+
       if (controller.tenants.isEmpty) {
         return Center(
           child: MyText(
@@ -208,12 +215,8 @@ class TenantCardList extends StatelessWidget {
           ),
         );
       }
-      if (controller.tenants.length == 1) {
-        return const SizedBox.shrink();
-      }
-      return SingleChildScrollView(
-        padding: const EdgeInsets.symmetric(horizontal: 0, vertical: 24),
-        child: Column(
+
+      return Column(
         crossAxisAlignment: CrossAxisAlignment.center,
         children: [
           ...controller.tenants.map(
@@ -225,8 +228,8 @@ class TenantCardList extends StatelessWidget {
           const SizedBox(height: 16),
           TextButton.icon(
             onPressed: () => Get.back(),
-              icon: const Icon(Icons.arrow_back,
-                  size: 20, color: Colors.redAccent),
+            icon:
+                const Icon(Icons.arrow_back, size: 20, color: Colors.redAccent),
             label: MyText(
               'Back to Login',
               color: Colors.red,
@@ -235,12 +238,12 @@ class TenantCardList extends StatelessWidget {
             ),
           ),
         ],
-        ),
       );
     });
   }
 }
 
+/// Single Tenant Card
 class _TenantCard extends StatelessWidget {
   final dynamic tenant;
   final VoidCallback onTap;
@@ -253,9 +256,7 @@ class _TenantCard extends StatelessWidget {
       borderRadius: BorderRadius.circular(5),
       child: Card(
         elevation: 3,
-        shape: RoundedRectangleBorder(
-          borderRadius: BorderRadius.circular(5),
-        ),
+        shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(5)),
         margin: const EdgeInsets.only(bottom: 20),
         child: Padding(
           padding: const EdgeInsets.all(16),
@@ -291,11 +292,7 @@ class _TenantCard extends StatelessWidget {
                   ],
                 ),
               ),
-              Icon(
-                Icons.arrow_forward_ios,
-                size: 24,
-                color: Colors.red,
-              ),
+              const Icon(Icons.arrow_forward_ios, size: 24, color: Colors.red),
             ],
           ),
         ),
@@ -304,6 +301,7 @@ class _TenantCard extends StatelessWidget {
   }
 }
 
+/// Tenant Logo (supports base64 and URL)
 class TenantLogo extends StatelessWidget {
   final String? logoImage;
   const TenantLogo({required this.logoImage});
@@ -311,9 +309,7 @@ class TenantLogo extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     if (logoImage == null || logoImage!.isEmpty) {
-      return Center(
-        child: Icon(Icons.business, color: Colors.grey.shade600),
-      );
+      return Center(child: Icon(Icons.business, color: Colors.grey.shade600));
     }
     if (logoImage!.startsWith("data:image")) {
       try {
@@ -321,9 +317,7 @@ class TenantLogo extends StatelessWidget {
         final bytes = base64Decode(base64Str);
         return Image.memory(bytes, fit: BoxFit.cover);
       } catch (_) {
-        return Center(
-          child: Icon(Icons.business, color: Colors.grey.shade600),
-        );
+        return Center(child: Icon(Icons.business, color: Colors.grey.shade600));
       }
     } else {
       return Image.network(
@@ -337,6 +331,7 @@ class TenantLogo extends StatelessWidget {
   }
 }
 
+/// Red Wave Background
 class _RedWaveBackground extends StatelessWidget {
   final Color brandRed;
   const _RedWaveBackground({required this.brandRed});
@@ -352,7 +347,6 @@ class _RedWaveBackground extends StatelessWidget {
 
 class _WavePainter extends CustomPainter {
   final Color brandRed;
-
   _WavePainter(this.brandRed);
 
   @override