refactor: extract device fetching logic into a separate method for improved readability

refactor: rename tab change events and states for consistency, and to match blocs naming convention.
implement SceneDevicesBodyTabBar for improved tab management in SceneDevicesBody.
2026-03-11 04:21:44 +00:00 · 2025-04-16 16:10:36 +03:00 · 2025-04-16 16:07:30 +03:00 · 2025-04-16 15:57:21 +03:00 · 2025-04-16 15:46:56 +03:00 · 2025-04-16 15:25:23 +03:00
19 changed files with 488 additions and 545 deletions
--- a/.env.dev
+++ b/.env.dev
@ -1,3 +1,5 @@
 ENV_NAME=development
 BASE_URL=https://syncrow-dev.azurewebsites.net
 PROJECT_ID=0e62577c-06fa-41b9-8a92-99a21fbaf51c
+CLIENT_ID=c024573d191a327ce74db7d4502fdc29
+CLIENT_SECRET=fec6ccbc33664f94a3b84a45c7cceef3f3ebd1613ef4307db8946ede530cd1ed
--- a/lib/features/auth/bloc/auth_cubit.dart
+++ b/lib/features/auth/bloc/auth_cubit.dart
@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:flutter_dotenv/flutter_dotenv.dart';
 import 'package:flutter_secure_storage/flutter_secure_storage.dart';
 import 'package:syncrow_app/features/auth/model/login_with_email_model.dart';
 import 'package:syncrow_app/features/auth/model/signup_model.dart';
@ -217,9 +218,22 @@ class AuthCubit extends Cubit<AuthState> {
  signUp() async {
    emit(AuthLoginLoading());
    final response;
+
+    final clientId = dotenv.env['CLIENT_ID'] ?? '';
+    final clientSecret = dotenv.env['CLIENT_SECRET'] ?? '';
+
    try {
      List<String> userFullName = fullName.split(' ');
+
+      final clientToken = await AuthenticationAPI.fetchClientToken(
+        clientId: clientId,
+        clientSecret: clientSecret,
+      );
+
+      final accessToken = clientToken['accessToken'];
+
      response = await AuthenticationAPI.signUp(
+        accessToken: accessToken,
        model: SignUpModel(
            hasAcceptedAppAgreement: true,
            email: email.toLowerCase(),
--- a/lib/features/devices/bloc/device_manager_bloc/device_manager_bloc.dart
+++ b/lib/features/devices/bloc/device_manager_bloc/device_manager_bloc.dart
@ -169,19 +169,18 @@ class DeviceManagerBloc extends Bloc<DeviceManagerEvent, DeviceManagerState> {
    }
  }

-  List<DeviceModel> _getOnlyImplementedDevices(List<DeviceModel> devices) {
-    const allowedDeviceTypes = {
-      DeviceType.AC,
-      DeviceType.DoorLock,
-      DeviceType.Gateway,
-      DeviceType.WallSensor,
-      DeviceType.CeilingSensor,
-      DeviceType.ThreeGang,
-      DeviceType.OneGang,
-    };
-
-    return devices
-        .where((device) => allowedDeviceTypes.contains(device.productType))
-        .toList();
+  _getOnlyImplementedDevices(List<DeviceModel> devices) {
+    List<DeviceModel> implementedDevices = [];
+    for (int i = 0; i < devices.length; i++) {
+      if (devices[i].productType == DeviceType.AC ||
+          devices[i].productType == DeviceType.DoorLock ||
+          devices[i].productType == DeviceType.Gateway ||
+          devices[i].productType == DeviceType.WallSensor ||
+          devices[i].productType == DeviceType.CeilingSensor ||
+          devices[i].productType == DeviceType.ThreeGang) {
+        implementedDevices.add(devices[i]);
+      }
+    }
+    return implementedDevices;
  }
 }
--- a/lib/features/scene/bloc/tab_change/tab_change_bloc.dart
+++ b/lib/features/scene/bloc/tab_change/tab_change_bloc.dart
@ -1,5 +1,3 @@
-import 'dart:async';
-
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:syncrow_app/features/devices/bloc/device_manager_bloc/device_manager_bloc.dart';
 import 'package:syncrow_app/features/devices/bloc/device_manager_bloc/device_manager_event.dart';
@ -7,19 +5,31 @@ import 'package:syncrow_app/features/scene/bloc/tab_change/tab_change_event.dart
 import 'package:syncrow_app/features/scene/bloc/tab_change/tab_change_state.dart';

 class TabBarBloc extends Bloc<TabBarEvent, TabBarState> {
-  final DeviceManagerBloc deviceManagerBloc;
-  TabBarBloc(this.deviceManagerBloc) : super(const Initial()) {
-    on<TabChanged>(_handleTabChanged);
+  TabBarBloc(this.deviceManagerBloc) : super(const TabBarInitialState()) {
+    on<TabBarTabChangedEvent>(_onTabBarTabChangedEvent);
  }

-  FutureOr<void> _handleTabChanged(
-      TabChanged event, Emitter<TabBarState> emit) {
+  final DeviceManagerBloc deviceManagerBloc;
+
+  void _onTabBarTabChangedEvent(
+    TabBarTabChangedEvent event,
+    Emitter<TabBarState> emit,
+  ) {
+    _getDevices(event);
+
+    emit(
+      TabBarTabSelectedState(
+        roomId: event.roomId,
+        selectedTabIndex: event.selectedIndex,
+      ),
+    );
+  }
+
+  void _getDevices(TabBarTabChangedEvent event) {
    if (event.roomId == "-1") {
      deviceManagerBloc.add(FetchAllDevices());
    } else {
      deviceManagerBloc.add(FetchDevicesByRoomId(event.roomId, event.unit));
    }
-    emit(TabSelected(
-        roomId: event.roomId, selectedTabIndex: event.selectedIndex));
  }
 }
--- a/lib/features/scene/bloc/tab_change/tab_change_event.dart
+++ b/lib/features/scene/bloc/tab_change/tab_change_event.dart
@ -4,10 +4,14 @@ abstract class TabBarEvent {
  const TabBarEvent();
 }

-class TabChanged extends TabBarEvent {
+class TabBarTabChangedEvent extends TabBarEvent {
+  const TabBarTabChangedEvent({
+    required this.selectedIndex,
+    required this.roomId,
+    required this.unit,
+  });
+
  final int selectedIndex;
  final String roomId;
  final SpaceModel unit;
-  const TabChanged(
-      {required this.selectedIndex, required this.roomId, required this.unit});
 }
--- a/lib/features/scene/bloc/tab_change/tab_change_state.dart
+++ b/lib/features/scene/bloc/tab_change/tab_change_state.dart
@ -2,12 +2,16 @@ abstract class TabBarState {
  const TabBarState();
 }

-class Initial extends TabBarState {
-  const Initial();
+class TabBarInitialState extends TabBarState {
+  const TabBarInitialState();
 }

-class TabSelected extends TabBarState {
+class TabBarTabSelectedState extends TabBarState {
+  const TabBarTabSelectedState({
+    required this.roomId,
+    required this.selectedTabIndex,
+  });
+
  final int selectedTabIndex;
  final String roomId;
-  const TabSelected({required this.roomId, required this.selectedTabIndex});
 }
--- a/lib/features/scene/helper/functions_per_device/one_gang_functions.dart
+++ b/lib/features/scene/helper/functions_per_device/one_gang_functions.dart
@ -1,78 +0,0 @@
-import 'package:syncrow_app/features/scene/enum/operation_dialog_type.dart';
-import 'package:syncrow_app/features/scene/model/scene_static_function.dart';
-import 'package:syncrow_app/generated/assets.dart';
-
-class OneGangHelperFunctions {
-  static List<SceneStaticFunction> oneGangHelperFunctions(
-      String deviceId, String deviceName, functionValue) {
-    return [
-      SceneStaticFunction(
-        deviceId: deviceId,
-        deviceName: deviceName,
-        icon: Assets.assetsAcPower,
-        operationName: 'Light Switch',
-        code: 'switch_1',
-        functionValue: functionValue,
-        operationDialogType: OperationDialogType.onOff,
-        operationalValues: [
-          SceneOperationalValue(
-              icon: Assets.assetsAcPower, description: "ON", value: true),
-          SceneOperationalValue(
-              icon: Assets.assetsAcPowerOFF, description: "OFF", value: false),
-        ],
-      ),
-      SceneStaticFunction(
-        deviceId: deviceId,
-        deviceName: deviceName,
-        icon: Assets.assetsLightCountdown,
-        operationName: 'Light CountDown',
-        code: 'countdown_1',
-        functionValue: functionValue,
-        operationDialogType: OperationDialogType.countdown,
-        operationalValues: [
-          SceneOperationalValue(icon: '', value: 0),
-        ],
-      ),
-    ];
-  }
-
-  static List<SceneStaticFunction> oneGangAutomationFunctions(
-      String deviceId, String deviceName, functionValue) {
-    return [
-      SceneStaticFunction(
-        deviceId: deviceId,
-        deviceName: deviceName,
-        icon: Assets.assetsAcPower,
-        operationName: 'Light Switch',
-        code: 'switch_1',
-        functionValue: functionValue,
-        operationDialogType: OperationDialogType.onOff,
-        operationalValues: [
-          SceneOperationalValue(
-              icon: Assets.assetsAcPower, description: "ON", value: true),
-          SceneOperationalValue(
-              icon: Assets.assetsAcPowerOFF, description: "OFF", value: false),
-        ],
-      ),
-      SceneStaticFunction(
-        deviceId: deviceId,
-        deviceName: deviceName,
-        icon: Assets.assetsLightCountdown,
-        operationName: 'Light CountDown',
-        code: 'countdown_1',
-        functionValue: functionValue,
-        operationDialogType: OperationDialogType.integerSteps,
-        operationalValues: [
-          SceneOperationalValue(
-            icon: '',
-            description: "sec",
-            value: 0.0,
-            minValue: 0,
-            maxValue: 43200,
-            stepValue: 1,
-          ),
-        ],
-      ),
-    ];
-  }
-}
--- a/lib/features/scene/helper/scene_operations_data_helper.dart
+++ b/lib/features/scene/helper/scene_operations_data_helper.dart
@ -6,7 +6,6 @@ import 'package:syncrow_app/features/scene/helper/functions_per_device/ac_functi
 import 'package:syncrow_app/features/scene/helper/functions_per_device/door_lock_functions.dart';
 import 'package:syncrow_app/features/scene/helper/functions_per_device/gateway_functions.dart';
 import 'package:syncrow_app/features/scene/helper/functions_per_device/human_presence_functions.dart';
-import 'package:syncrow_app/features/scene/helper/functions_per_device/one_gang_functions.dart';
 import 'package:syncrow_app/features/scene/helper/functions_per_device/presence_sensor.dart';
 import 'package:syncrow_app/features/scene/helper/functions_per_device/three_gang_functions.dart';
 import 'package:syncrow_app/features/scene/model/scene_details_model.dart';
@ -15,9 +14,8 @@ import 'package:syncrow_app/generated/assets.dart';
 import 'package:syncrow_app/utils/resource_manager/constants.dart';

 mixin SceneOperationsDataHelper {
-  final Map<DeviceType,
-          Function(List<FunctionModel>, String, String, dynamic, bool)>
-      _functionMap = {
+  final Map<DeviceType, Function(List<FunctionModel>, String, String, dynamic, bool)> _functionMap =
+      {
    DeviceType.LightBulb: lightBulbFunctions,
    DeviceType.CeilingSensor: ceilingSensorFunctions,
    DeviceType.WallSensor: wallSensorFunctions,
@ -26,7 +24,6 @@ mixin SceneOperationsDataHelper {
    DeviceType.Curtain: curtainFunctions,
    DeviceType.ThreeGang: threeGangFunctions,
    DeviceType.Gateway: gatewayFunctions,
-    DeviceType.OneGang: oneGangFunctions,
  };

  final Map<DeviceType, String> _titleMap = {
@ -38,24 +35,7 @@ mixin SceneOperationsDataHelper {
    DeviceType.Curtain: 'Curtain Functions',
    DeviceType.ThreeGang: '3G Light Switch Functions',
    DeviceType.Gateway: 'Gateway Functions',
-    DeviceType.OneGang: '1G Light Switch Conditions',
  };
-  static String _productTypeCache = '';
-
-  //one gang functions
-  static List<SceneStaticFunction> oneGangFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
-    if (isAutomation) {
-      return OneGangHelperFunctions.oneGangAutomationFunctions(
-          deviceId, deviceName, functionValue);
-    }
-    return OneGangHelperFunctions.oneGangHelperFunctions(
-        deviceId, deviceName, functionValue);
-  }

  List<SceneStaticFunction> getFunctionsWithIcons({
    DeviceType? type,
@ -65,22 +45,16 @@ mixin SceneOperationsDataHelper {
    required bool isAutomation,
  }) {
    final functionValue = null;
-    return _functionMap[type]?.call(
-            functions, deviceId, deviceName, functionValue, isAutomation) ??
-        lightBulbFunctions(
-            functions, deviceId, deviceName, functionValue, isAutomation);
+    return _functionMap[type]?.call(functions, deviceId, deviceName, functionValue, isAutomation) ??
+        lightBulbFunctions(functions, deviceId, deviceName, functionValue, isAutomation);
  }

  String getTitle({DeviceType? type}) {
    return _titleMap[type] ?? '';
  }

-  static List<SceneStaticFunction> ceilingSensorFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
+  static List<SceneStaticFunction> ceilingSensorFunctions(List<FunctionModel> functions,
+      String deviceId, String deviceName, dynamic functionValue, bool isAutomation) {
    if (isAutomation) {
      return PresenceSensorHelperFunctions.automationPresenceSensorFunctions(
          deviceId, deviceName, functionValue);
@ -89,35 +63,22 @@ mixin SceneOperationsDataHelper {
        deviceId, deviceName, functionValue);
  }

-  static List<SceneStaticFunction> curtainFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
+  static List<SceneStaticFunction> curtainFunctions(List<FunctionModel> functions, String deviceId,
+      String deviceName, dynamic functionValue, bool isAutomation) {
    return [];
  }

-  static List<SceneStaticFunction> doorLockFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
+  static List<SceneStaticFunction> doorLockFunctions(List<FunctionModel> functions, String deviceId,
+      String deviceName, dynamic functionValue, bool isAutomation) {
    if (isAutomation) {
      return DoorLockHelperFunctions.doorLockAutomationFunctions(
          deviceId, deviceName, functionValue);
    }
-    return DoorLockHelperFunctions.doorLockTapToRunFunctions(
-        deviceId, deviceName, functionValue);
+    return DoorLockHelperFunctions.doorLockTapToRunFunctions(deviceId, deviceName, functionValue);
  }

-  static List<SceneStaticFunction> wallSensorFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
+  static List<SceneStaticFunction> wallSensorFunctions(List<FunctionModel> functions,
+      String deviceId, String deviceName, dynamic functionValue, bool isAutomation) {
    if (isAutomation) {
      return HumanPresenceHelperFunctions.automationHumanPresenceFunctions(
          deviceId, deviceName, functionValue);
@ -126,51 +87,31 @@ mixin SceneOperationsDataHelper {
        deviceId, deviceName, functionValue);
  }

-  static List<SceneStaticFunction> lightBulbFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
+  static List<SceneStaticFunction> lightBulbFunctions(List<FunctionModel> functions,
+      String deviceId, String deviceName, dynamic functionValue, bool isAutomation) {
    return [];
  }

-  static List<SceneStaticFunction> gatewayFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
-    return GatewayHelperFunctions.tabToRunGatewayFunctions(
-        deviceId, deviceName, functionValue);
+  static List<SceneStaticFunction> gatewayFunctions(List<FunctionModel> functions, String deviceId,
+      String deviceName, dynamic functionValue, bool isAutomation) {
+    return GatewayHelperFunctions.tabToRunGatewayFunctions(deviceId, deviceName, functionValue);
  }

-  static List<SceneStaticFunction> threeGangFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
+  static List<SceneStaticFunction> threeGangFunctions(List<FunctionModel> functions,
+      String deviceId, String deviceName, dynamic functionValue, bool isAutomation) {
    if (isAutomation) {
      return ThreeGangHelperFunctions.threeGangAutomationFunctions(
          deviceId, deviceName, functionValue);
    }
-    return ThreeGangHelperFunctions.threeGangHelperFunctions(
-        deviceId, deviceName, functionValue);
+    return ThreeGangHelperFunctions.threeGangHelperFunctions(deviceId, deviceName, functionValue);
  }

-  static List<SceneStaticFunction> acFunctions(
-      List<FunctionModel> functions,
-      String deviceId,
-      String deviceName,
-      dynamic functionValue,
-      bool isAutomation) {
+  static List<SceneStaticFunction> acFunctions(List<FunctionModel> functions, String deviceId,
+      String deviceName, dynamic functionValue, bool isAutomation) {
    if (isAutomation) {
-      return ACFunctionsHelper.automationAcFunctions(
-          deviceId, deviceName, functionValue);
+      return ACFunctionsHelper.automationAcFunctions(deviceId, deviceName, functionValue);
    }
-    return ACFunctionsHelper.tabToRunAcFunctions(
-        deviceId, deviceName, functionValue);
+    return ACFunctionsHelper.tabToRunAcFunctions(deviceId, deviceName, functionValue);
  }

  List<SceneStaticFunction> getTaskListFunctionsFromApi({
@ -208,12 +149,8 @@ mixin SceneOperationsDataHelper {
          SceneStaticFunction(
            deviceId: action.entityId,
            deviceName: action.name.toString(),
-            deviceIcon: action.type == 'automation'
-                ? Assets.player
-                : Assets.handClickIcon,
-            icon: action.type == 'automation'
-                ? Assets.player
-                : Assets.handClickIcon,
+            deviceIcon: action.type == 'automation' ? Assets.player : Assets.handClickIcon,
+            icon: action.type == 'automation' ? Assets.player : Assets.handClickIcon,
            operationName: action.type.toString(),
            operationDialogType: OperationDialogType.onOff,
            functionValue: action.actionExecutor,
@ -233,8 +170,7 @@ mixin SceneOperationsDataHelper {
          ),
        );
      } else {
-        functions
-            .add(_mapExecutorPropertyToSceneFunction(action, isAutomation));
+        functions.add(_mapExecutorPropertyToSceneFunction(action, isAutomation));
      }
    }

@ -270,9 +206,7 @@ mixin SceneOperationsDataHelper {
  }) {
    final executorProperty = action.executorProperty;

-    final Map<String,
-            SceneStaticFunction Function(Action, bool, String?, String?)>
-        functionMap = {
+    final Map<String, SceneStaticFunction Function(Action, bool, String?, String?)> functionMap = {
      'sensitivity': _createSensitivityFunction,
      'normal_open_switch': _createNormalOpenSwitchFunction,
      'unlock_fingerprint': _createUnlockFingerprintFunction,
@ -348,16 +282,14 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createSensitivityFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createSensitivityFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Presence Sensor',
      Assets.assetsIconsSensors,
      'Sensitivity',
-      isAutomation
-          ? OperationDialogType.integerSteps
-          : OperationDialogType.listOfOptions,
+      isAutomation ? OperationDialogType.integerSteps : OperationDialogType.listOfOptions,
      isAutomation ? _createIntegerStepsOptions() : _createSensitivityOptions(),
      isAutomation,
      comparator,
@ -365,8 +297,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createNormalOpenSwitchFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createNormalOpenSwitchFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -419,8 +351,8 @@ mixin SceneOperationsDataHelper {
    ];
  }

-  SceneStaticFunction _createUnlockFingerprintFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createUnlockFingerprintFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -434,8 +366,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createUnlockPasswordFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createUnlockPasswordFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -449,8 +381,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createUnlockCardFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createUnlockCardFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -464,8 +396,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createAlarmLockFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createAlarmLockFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -479,8 +411,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createUnlockRequestFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createUnlockRequestFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -494,8 +426,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createResidualElectricityFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createResidualElectricityFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -509,8 +441,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createReverseLockFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createReverseLockFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -524,8 +456,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createUnlockAppFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createUnlockAppFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -539,8 +471,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createHijackFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createHijackFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -554,8 +486,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createDoorbellFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createDoorbellFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -569,8 +501,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createUnlockTemporaryFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createUnlockTemporaryFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'WIFI LOCK PRO',
@ -584,8 +516,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createFarDetectionFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createFarDetectionFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -599,8 +531,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createMotionSensitivityFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createMotionSensitivityFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -614,8 +546,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createMotionlessSensitivityFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createMotionlessSensitivityFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -629,8 +561,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createIndicatorFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createIndicatorFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -644,8 +576,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createPresenceTimeFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createPresenceTimeFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -659,8 +591,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createPresenceStateFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createPresenceStateFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -674,16 +606,14 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createDisCurrentFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createDisCurrentFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
      Assets.assetsIconsSensors,
      'Current Distance',
-      isAutomation
-          ? OperationDialogType.integerSteps
-          : OperationDialogType.countdown,
+      isAutomation ? OperationDialogType.integerSteps : OperationDialogType.countdown,
      _createCurrentDistanceOptions(),
      isAutomation,
      comparator,
@ -691,8 +621,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createIlluminanceValueFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createIlluminanceValueFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -706,8 +636,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createCheckingResultFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createCheckingResultFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Human Presence Sensor',
@ -721,8 +651,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createSwitchFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createSwitchFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Smart AC Thermostat - Grey - Model A',
@ -736,27 +666,23 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createTempSetFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createTempSetFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Smart AC Thermostat - Grey - Model A',
      Assets.assetsIconsAC,
      'Set Temperature',
-      isAutomation
-          ? OperationDialogType.integerSteps
-          : OperationDialogType.temperature,
-      isAutomation
-          ? _createAutomationTemperatureOptions()
-          : _createTemperatureOptions(),
+      isAutomation ? OperationDialogType.integerSteps : OperationDialogType.temperature,
+      isAutomation ? _createAutomationTemperatureOptions() : _createTemperatureOptions(),
      isAutomation,
      comparator,
      uniqueCustomId,
    );
  }

-  SceneStaticFunction _createTempCurrentFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createTempCurrentFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Smart AC Thermostat - Grey - Model A',
@ -770,8 +696,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createModeFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createModeFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Smart AC Thermostat - Grey - Model A',
@ -785,8 +711,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createLevelFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createLevelFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Smart AC Thermostat - Grey - Model A',
@ -800,8 +726,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createChildLockFunction(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createChildLockFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Smart AC Thermostat - Grey - Model A',
@ -815,10 +741,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createSwitch1Function(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
-    switch (action.productType) {
-      case "3G":
+  SceneStaticFunction _createSwitch1Function(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      '3 Gang Button Switch L-L',
@ -830,23 +754,10 @@ mixin SceneOperationsDataHelper {
      comparator,
      uniqueCustomId,
    );
-      default:
-        return _createSceneFunction(
-          action,
-          '1 Gang Button Switch L-L',
-          Assets.oneGang,
-          'Light Switch',
-          OperationDialogType.onOff,
-          _createOnOffOptions(),
-          isAutomation,
-          comparator,
-          uniqueCustomId,
-        );
-    }
  }

-  SceneStaticFunction _createSwitch2Function(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createSwitch2Function(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      '3 Gang Button Switch L-L',
@ -860,8 +771,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createSwitch3Function(Action action, bool isAutomation,
-      String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createSwitch3Function(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      '3 Gang Button Switch L-L',
@ -875,84 +786,53 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createCountdown1Function(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
-    switch (action.productType) {
-      case "3G":
+  SceneStaticFunction _createCountdown1Function(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      '3 Gang Button Switch L-L',
      Assets.assetsIcons3GangSwitch,
      'Light 1 CountDown',
-          isAutomation
-              ? OperationDialogType.integerSteps
-              : OperationDialogType.countdown,
-          isAutomation
-              ? _createAutomationCountDownOptions()
-              : _createCountdownOptions(),
+      isAutomation ? OperationDialogType.integerSteps : OperationDialogType.countdown,
+      isAutomation ? _createAutomationCountDownOptions() : _createCountdownOptions(),
      isAutomation,
      comparator,
      uniqueCustomId,
    );
-      default:
-        return _createSceneFunction(
-          action,
-          '1 Gang Button Switch L-L',
-          Assets.oneGang,
-          'Light CountDown',
-          isAutomation
-              ? OperationDialogType.integerSteps
-              : OperationDialogType.countdown,
-          isAutomation
-              ? _createAutomationCountDownOptions()
-              : _createCountdownOptions(),
-          isAutomation,
-          comparator,
-          uniqueCustomId,
-        );
-    }
  }

-  SceneStaticFunction _createCountdown2Function(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createCountdown2Function(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      '3 Gang Button Switch L-L',
      Assets.assetsIcons3GangSwitch,
      'Light 2 CountDown',
-      isAutomation
-          ? OperationDialogType.integerSteps
-          : OperationDialogType.countdown,
-      isAutomation
-          ? _createAutomationCountDownOptions()
-          : _createCountdownOptions(),
+      isAutomation ? OperationDialogType.integerSteps : OperationDialogType.countdown,
+      isAutomation ? _createAutomationCountDownOptions() : _createCountdownOptions(),
      isAutomation,
      comparator,
      uniqueCustomId,
    );
  }

-  SceneStaticFunction _createCountdown3Function(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createCountdown3Function(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      '3 Gang Button Switch L-L',
      Assets.assetsIcons3GangSwitch,
      'Light 3 CountDown',
-      isAutomation
-          ? OperationDialogType.integerSteps
-          : OperationDialogType.countdown,
-      isAutomation
-          ? _createAutomationCountDownOptions()
-          : _createCountdownOptions(),
+      isAutomation ? OperationDialogType.integerSteps : OperationDialogType.countdown,
+      isAutomation ? _createAutomationCountDownOptions() : _createCountdownOptions(),
      isAutomation,
      comparator,
      uniqueCustomId,
    );
  }

-  SceneStaticFunction _createSwitchAlarmSoundFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createSwitchAlarmSoundFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Gateway',
@ -966,8 +846,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createMasterStateFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createMasterStateFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Gateway',
@ -981,8 +861,8 @@ mixin SceneOperationsDataHelper {
    );
  }

-  SceneStaticFunction _createFactoryResetFunction(Action action,
-      bool isAutomation, String? comparator, String? uniqueCustomId) {
+  SceneStaticFunction _createFactoryResetFunction(
+      Action action, bool isAutomation, String? comparator, String? uniqueCustomId) {
    return _createSceneFunction(
      action,
      'Gateway',
@ -1300,12 +1180,8 @@ mixin SceneOperationsDataHelper {
          uniqueCustomId: taskItem.uniqueCustomId,
          deviceId: taskItem.deviceId,
          deviceName: taskItem.deviceName.toString(),
-          deviceIcon: taskItem.operationName == 'automation'
-              ? Assets.player
-              : Assets.handClickIcon,
-          icon: taskItem.operationName == 'automation'
-              ? Assets.player
-              : Assets.handClickIcon,
+          deviceIcon: taskItem.operationName == 'automation' ? Assets.player : Assets.handClickIcon,
+          icon: taskItem.operationName == 'automation' ? Assets.player : Assets.handClickIcon,
          operationName: taskItem.operationName,
          operationDialogType: OperationDialogType.onOff,
          functionValue: taskItem.functionValue == 'rule_enable' ? true : false,
--- a/lib/features/scene/model/scene_details_model.dart
+++ b/lib/features/scene/model/scene_details_model.dart
@ -74,7 +74,6 @@ class Action {
  ExecutorProperty? executorProperty;
  String? name;
  String? type;
-  String? productType;

  Action({
    required this.actionExecutor,
@ -82,7 +81,6 @@ class Action {
    this.executorProperty,
    this.name,
    this.type,
-    this.productType,
  });

  String toRawJson() => json.encode(toJson());
@ -90,11 +88,10 @@ class Action {
  static Action? fromJson(Map<String, dynamic> json) {
    if (json['name'] != null && json['type'] != null) {
      return Action(
-        actionExecutor: json["actionExecutor"] as String,
-        entityId: json["entityId"] as String,
-        name: json['name'] as String?,
-        type: json['type'] as String?,
-        productType: json['productType'] as String?,
+        actionExecutor: json["actionExecutor"],
+        entityId: json["entityId"],
+        name: json['name'],
+        type: json['type'],
      );
    }
    if (json["executorProperty"] == null) {
@ -102,10 +99,9 @@ class Action {
    }

    return Action(
-      actionExecutor: json["actionExecutor"] as String,
-      entityId: json["entityId"] as String,
+      actionExecutor: json["actionExecutor"],
+      entityId: json["entityId"],
      executorProperty: ExecutorProperty.fromJson(json["executorProperty"]),
-      productType: json['productType'] as String?,
    );
  }

--- a/lib/features/scene/view/scene_rooms_tabbar.dart
+++ b/lib/features/scene/view/scene_rooms_tabbar.dart
@ -20,82 +20,69 @@ class SceneRoomsTabBarDevicesView extends StatefulWidget {
      _SceneRoomsTabBarDevicesViewState();
 }

-class _SceneRoomsTabBarDevicesViewState
-    extends State<SceneRoomsTabBarDevicesView>
+class _SceneRoomsTabBarDevicesViewState extends State<SceneRoomsTabBarDevicesView>
    with SingleTickerProviderStateMixin {
  late final TabController _tabController;
-  List<SubSpaceModel>? rooms = [];
  late final SpaceModel selectedSpace;
+  var rooms = <SubSpaceModel>[];

  @override
  void initState() {
    super.initState();

-    WidgetsBinding.instance.addPostFrameCallback((_) {
    selectedSpace = HomeCubit.getInstance().selectedSpace!;

-      rooms = List.from(selectedSpace.subspaces ?? []);
-
-      if (rooms != null && rooms!.isNotEmpty) {
-        if (rooms![0].id != '-1') {
-          rooms?.insert(
-            0,
-            SubSpaceModel(
+    rooms = List.from(selectedSpace.subspaces);
+    final defaultSubSpaceModel = SubSpaceModel(
      name: 'All Devices',
      devices: context.read<DevicesCubit>().allDevices,
      id: '-1',
-            ),
    );
+
+    if (rooms.isNotEmpty) {
+      final isFirstRoomIdValid = rooms[0].id != '-1';
+      if (isFirstRoomIdValid) {
+        rooms.insert(0, defaultSubSpaceModel);
      }
    } else {
-        rooms = [
-          SubSpaceModel(
-            name: 'All Devices',
-            devices: context.read<DevicesCubit>().allDevices,
-            id: '-1',
-          )
-        ];
+      rooms = [defaultSubSpaceModel];
    }

-      _tabController = TabController(length: rooms!.length, vsync: this);
+    _tabController = TabController(length: rooms.length, vsync: this);
    _tabController.addListener(_handleTabSwitched);
-      setState(() {});
-    });
-  }
-
-  void _handleTabSwitched() {
-    if (_tabController.indexIsChanging) {
-      final value = _tabController.index;
-
-      /// select tab
-      context.read<TabBarBloc>().add(TabChanged(
-          selectedIndex: value,
-          roomId: rooms?[value].id ?? '',
-          unit: selectedSpace));
-      return;
-    }
  }

  @override
  void dispose() {
    super.dispose();
-    _tabController.dispose();
    _tabController.removeListener(() {});
+    _tabController.dispose();
+  }
+
+  void _handleTabSwitched() {
+    if (_tabController.indexIsChanging) {
+      final index = _tabController.index;
+
+      context.read<TabBarBloc>().add(
+            TabBarTabChangedEvent(
+              selectedIndex: index,
+              roomId: rooms[index].id ?? '',
+              unit: selectedSpace,
+            ),
+          );
+      return;
+    }
  }

  @override
  Widget build(BuildContext context) {
    return DefaultScaffold(
-      title: StringsManager.createScene,
-      padding: EdgeInsets.zero,
+      padding: EdgeInsetsDirectional.zero,
      leading: IconButton(
-        onPressed: () {
-          navigateToRoute(context, Routes.sceneTasksRoute);
-        },
-        icon: const Icon(
-          Icons.arrow_back_ios,
-        ),
+        onPressed: () => navigateToRoute(context, Routes.sceneTasksRoute),
+        icon: const Icon(Icons.arrow_back_ios),
      ),
+      title: StringsManager.createScene,
      child: SceneDevicesBody(tabController: _tabController, rooms: rooms),
    );
  }
--- a/lib/features/scene/widgets/scene_devices/scene_devices_body.dart
+++ b/lib/features/scene/widgets/scene_devices/scene_devices_body.dart
@ -1,6 +1,5 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:flutter_svg/flutter_svg.dart';
 import 'package:syncrow_app/features/devices/bloc/device_manager_bloc/device_manager_bloc.dart';
 import 'package:syncrow_app/features/devices/bloc/device_manager_bloc/device_manager_state.dart';
 import 'package:syncrow_app/features/devices/model/subspace_model.dart';
@ -8,60 +7,44 @@ import 'package:syncrow_app/features/scene/bloc/tab_change/tab_change_bloc.dart'
 import 'package:syncrow_app/features/scene/bloc/tab_change/tab_change_state.dart';
 import 'package:syncrow_app/features/scene/enum/create_scene_enum.dart';
 import 'package:syncrow_app/features/scene/model/scene_settings_route_arguments.dart';
-import 'package:syncrow_app/features/scene/widgets/scene_list_tile.dart';
-import 'package:syncrow_app/features/shared_widgets/default_container.dart';
-import 'package:syncrow_app/features/shared_widgets/text_widgets/body_large.dart';
-import 'package:syncrow_app/features/shared_widgets/text_widgets/body_medium.dart';
-import 'package:syncrow_app/navigation/routing_constants.dart';
-import 'package:syncrow_app/utils/context_extension.dart';
-import 'package:syncrow_app/utils/resource_manager/color_manager.dart';
+import 'package:syncrow_app/features/scene/widgets/scene_devices/scene_devices_body_tab_bar.dart';
+import 'package:syncrow_app/features/scene/widgets/scene_devices/scene_devices_list.dart';
+import 'package:syncrow_app/features/shared_widgets/app_loading_indicator.dart';

 class SceneDevicesBody extends StatelessWidget {
  const SceneDevicesBody({
-    super.key,
-    required TabController tabController,
+    required this.tabController,
    required this.rooms,
-  }) : _tabController = tabController;
+    super.key,
+  });

-  final TabController _tabController;
-  final List<SubSpaceModel>? rooms;
+  final TabController tabController;
+  final List<SubSpaceModel> rooms;

  @override
  Widget build(BuildContext context) {
-    final isAutomationDeviceStatus =
-        ((ModalRoute.of(context)?.settings.arguments as SceneSettingsRouteArguments?)?.sceneType ==
-            CreateSceneEnum.deviceStatusChanges.name);
    return BlocBuilder<TabBarBloc, TabBarState>(
-      builder: (context, tabState) {
+      builder: (context, state) {
        return Column(
          mainAxisSize: MainAxisSize.min,
          children: [
-            TabBar(
-              controller: _tabController,
-              dividerColor: Colors.transparent,
-              indicatorColor: Colors.transparent,
-              tabs: [
-                ...rooms!.map((e) => Tab(
-                      child: BodyLarge(
-                        text: e.name ?? '',
-                        textAlign: TextAlign.start,
-                        style: context.bodyLarge.copyWith(
-                          color: (tabState is TabSelected) && tabState.roomId == e.id
-                              ? ColorsManager.textPrimaryColor
-                              : ColorsManager.textPrimaryColor.withOpacity(0.2),
-                        ),
-                      ),
-                    )),
-              ],
-              isScrollable: true,
-              tabAlignment: TabAlignment.start,
+            SceneDevicesBodyTabBar(
+              tabController: tabController,
+              rooms: rooms,
+              selectedRoomId: state is TabBarTabSelectedState ? state.roomId : '-1',
            ),
            Expanded(
              child: TabBarView(
-                controller: _tabController,
                physics: const NeverScrollableScrollPhysics(),
-                children:
-                    rooms!.map((e) => _buildRoomTab(e, context, isAutomationDeviceStatus)).toList(),
+                controller: tabController,
+                children: rooms
+                    .map(
+                      (room) => _buildRoomTab(
+                        room,
+                        _isAutomationDeviceStatus(context),
+                      ),
+                    )
+                    .toList(),
              ),
            ),
          ],
@ -70,52 +53,46 @@ class SceneDevicesBody extends StatelessWidget {
    );
  }

-  Widget _buildRoomTab(SubSpaceModel room, BuildContext context, bool isAutomationDeviceStatus) {
+  bool _isAutomationDeviceStatus(BuildContext context) {
+    final routeArguments =
+        ModalRoute.of(context)?.settings.arguments as SceneSettingsRouteArguments?;
+    final deviceStatusChangesScene = CreateSceneEnum.deviceStatusChanges.name;
+    final sceneType = routeArguments?.sceneType;
+
+    return sceneType == deviceStatusChangesScene;
+  }
+
+  Widget _buildRoomTab(
+    SubSpaceModel room,
+    bool isAutomationDeviceStatus,
+  ) {
    return BlocBuilder<DeviceManagerBloc, DeviceManagerState>(
      builder: (context, state) {
-        if (state.loading && state.devices == null) {
-          return const Center(child: CircularProgressIndicator());
-        } else if (state.devices != null && state.devices!.isNotEmpty) {
-          return ListView.builder(
-            itemCount: state.devices!.length,
-            itemBuilder: (context, index) {
-              final device = state.devices![index];
-              return DefaultContainer(
-                child: SceneListTile(
-                  minLeadingWidth: 40,
-                  leadingWidget: SvgPicture.asset(device.icon ?? ''),
-                  titleWidget: BodyMedium(
-                    text: device.name ?? '',
-                    style: context.titleSmall.copyWith(
-                      color: ColorsManager.secondaryTextColor,
-                      fontWeight: FontWeight.w400,
-                      fontSize: 20,
+        final isLoading = state.loading && state.devices == null;
+        final hasData =
+            state.devices != null && (state.devices?.isNotEmpty ?? false);
+        final hasError = state.error != null;
+
+        final widgets = <bool, Widget>{
+          isLoading: const AppLoadingIndicator(),
+          hasError: Center(child: Text('${state.error}')),
+          hasData: SceneDevicesList(
+            devices: state.devices ?? [],
+            isAutomationDeviceStatus: isAutomationDeviceStatus,
          ),
-                  ),
-                  trailingWidget: const Icon(
-                    Icons.arrow_forward_ios_rounded,
-                    color: ColorsManager.greyColor,
-                    size: 16,
-                  ),
-                  onPressed: () {
-                    Navigator.pushNamed(
-                      context,
-                      Routes.deviceFunctionsRoute,
-                      arguments: {
-                        "device": device,
-                        "isAutomationDeviceStatus": isAutomationDeviceStatus
-                      },
+        };
+
+        final invalidWidgetEntry = MapEntry(
+          true,
+          Center(child: Text('This subspace has no devices')),
        );
-                  },
-                ),
+
+        final validWidgetEntry = widgets.entries.firstWhere(
+          (entry) => entry.key == true,
+          orElse: () => invalidWidgetEntry,
        );
-            },
-          );
-        } else if (state.error != null) {
-          return const Center(child: Text('Something went wrong'));
-        } else {
-          return const SizedBox();
-        }
+
+        return validWidgetEntry.value;
      },
    );
  }
--- a/lib/features/scene/widgets/scene_devices/scene_devices_body_tab_bar.dart
+++ b/lib/features/scene/widgets/scene_devices/scene_devices_body_tab_bar.dart
@ -0,0 +1,46 @@
+import 'package:flutter/material.dart';
+import 'package:syncrow_app/features/devices/model/subspace_model.dart';
+import 'package:syncrow_app/features/shared_widgets/text_widgets/body_large.dart';
+import 'package:syncrow_app/utils/context_extension.dart';
+import 'package:syncrow_app/utils/resource_manager/color_manager.dart';
+
+class SceneDevicesBodyTabBar extends StatelessWidget {
+  const SceneDevicesBodyTabBar({
+    required this.tabController,
+    required this.rooms,
+    required this.selectedRoomId,
+    super.key,
+  });
+
+  final String selectedRoomId;
+
+  final TabController tabController;
+  final List<SubSpaceModel> rooms;
+
+  @override
+  Widget build(BuildContext context) {
+    return TabBar(
+      controller: tabController,
+      dividerColor: Colors.transparent,
+      indicatorColor: Colors.transparent,
+      isScrollable: true,
+      tabAlignment: TabAlignment.start,
+      tabs: rooms.map((e) {
+        final isSelected = selectedRoomId == e.id;
+        return Tab(
+          child: BodyLarge(
+            text: e.name ?? '',
+            textAlign: TextAlign.start,
+            style: context.bodyLarge.copyWith(
+              color: isSelected
+                  ? ColorsManager.textPrimaryColor
+                  : ColorsManager.textPrimaryColor.withValues(
+                      alpha: 0.2,
+                    ),
+            ),
+          ),
+        );
+      }).toList(),
+    );
+  }
+}
--- a/lib/features/scene/widgets/scene_devices/scene_devices_list.dart
+++ b/lib/features/scene/widgets/scene_devices/scene_devices_list.dart
@ -0,0 +1,66 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_svg/flutter_svg.dart';
+import 'package:syncrow_app/features/devices/model/device_model.dart';
+import 'package:syncrow_app/features/scene/widgets/scene_list_tile.dart';
+import 'package:syncrow_app/features/shared_widgets/default_container.dart';
+import 'package:syncrow_app/features/shared_widgets/text_widgets/body_medium.dart';
+import 'package:syncrow_app/navigation/routing_constants.dart';
+import 'package:syncrow_app/utils/context_extension.dart';
+import 'package:syncrow_app/utils/resource_manager/color_manager.dart';
+
+class SceneDevicesList extends StatelessWidget {
+  const SceneDevicesList({
+    required this.isAutomationDeviceStatus,
+    required this.devices,
+    super.key,
+  });
+  final List<DeviceModel> devices;
+  final bool isAutomationDeviceStatus;
+
+  @override
+  Widget build(BuildContext context) {
+    return ListView.builder(
+      itemCount: devices.length,
+      itemBuilder: (context, index) => _buildDeviceTile(context, devices[index]),
+    );
+  }
+
+  Widget _buildDeviceTile(
+    BuildContext context,
+    DeviceModel device,
+  ) {
+    return DefaultContainer(
+      child: SceneListTile(
+        minLeadingWidth: 40,
+        leadingWidget: SvgPicture.asset(device.icon ?? ''),
+        titleWidget: BodyMedium(
+          text: device.name ?? '',
+          style: context.titleSmall.copyWith(
+            color: ColorsManager.secondaryTextColor,
+            fontWeight: FontWeight.w400,
+            fontSize: 20,
+          ),
+        ),
+        trailingWidget: const Icon(
+          Icons.arrow_forward_ios_rounded,
+          color: ColorsManager.greyColor,
+          size: 16,
+        ),
+        onPressed: () => _navigateToDeviceFunctions(context, device),
+      ),
+    );
+  }
+
+  void _navigateToDeviceFunctions(
+    BuildContext context,
+    DeviceModel device,
+  ) {
+    Navigator.of(context).pushNamed(
+      Routes.deviceFunctionsRoute,
+      arguments: {
+        "device": device,
+        "isAutomationDeviceStatus": isAutomationDeviceStatus,
+      },
+    );
+  }
+}
--- a/lib/navigation/router.dart
+++ b/lib/navigation/router.dart
@ -3,8 +3,8 @@ import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:syncrow_app/features/app_layout/model/community_model.dart';
 import 'package:syncrow_app/features/app_layout/model/space_model.dart';
 import 'package:syncrow_app/features/app_layout/view/app_layout.dart';
-import 'package:syncrow_app/features/auth/view/otp_view.dart';
 import 'package:syncrow_app/features/auth/view/login_view.dart';
+import 'package:syncrow_app/features/auth/view/otp_view.dart';
 import 'package:syncrow_app/features/auth/view/sign_up_view.dart';
 import 'package:syncrow_app/features/dashboard/view/dashboard_view.dart';
 import 'package:syncrow_app/features/devices/bloc/device_manager_bloc/device_manager_bloc.dart';
@ -17,11 +17,12 @@ import 'package:syncrow_app/features/scene/bloc/tab_change/tab_change_bloc.dart'
 import 'package:syncrow_app/features/scene/bloc/tab_change/tab_change_event.dart';
 import 'package:syncrow_app/features/scene/view/device_functions_view.dart';
 import 'package:syncrow_app/features/scene/view/scene_auto_settings.dart';
-import 'package:syncrow_app/features/scene/view/scene_tasks_view.dart';
 import 'package:syncrow_app/features/scene/view/scene_rooms_tabbar.dart';
+import 'package:syncrow_app/features/scene/view/scene_tasks_view.dart';
 import 'package:syncrow_app/features/scene/view/scene_view.dart';
 import 'package:syncrow_app/features/scene/view/smart_automation_select_route.dart';
 import 'package:syncrow_app/features/splash/view/splash_view.dart';
+
 import 'routing_constants.dart';

 class Router {
@ -88,7 +89,7 @@ class Router {
                    BlocProvider(
                      create: (BuildContext context) =>
                          TabBarBloc(context.read<DeviceManagerBloc>())
-                            ..add(TabChanged(
+                            ..add(TabBarTabChangedEvent(
                                selectedIndex: 0,
                                roomId: '-1',
                                unit: SpaceModel(
--- a/lib/services/api/api_links_endpoints.dart
+++ b/lib/services/api/api_links_endpoints.dart
@ -11,7 +11,7 @@ abstract class ApiEndpoints {
  static const String sendOtp = '/authentication/user/send-otp';
  static const String verifyOtp = '/authentication/user/verify-otp';
  static const String forgetPassword = '/authentication/user/forget-password';
-
+  static const String clientLogin = 'client/token';
 ////////////////////////////////////// Spaces ///////////////////////////////////////

  ///Community Module
@ -111,9 +111,9 @@ abstract class ApiEndpoints {
 //POST
  static const String addDeviceToRoom = '/device/room';
  static const String addDeviceToGroup = '/device/group';
-  static const String controlDevice = '/device/{deviceUuid}/control';
+  static const String controlDevice = '/devices/{deviceUuid}/command';
  static const String firmwareDevice =
-      '/device/{deviceUuid}/firmware/{firmwareVersion}';
+      '/devices/{deviceUuid}/firmware/{firmwareVersion}';
  static const String getDevicesByUserId = '/device/user/{userId}';
  static const String getDevicesByUnitId = '/device/unit/{unitUuid}';
  static const String openDoorLock = '/door-lock/open/{doorLockUuid}';
@ -121,13 +121,13 @@ abstract class ApiEndpoints {
 //GET
  static const String deviceByRoom =
      '/projects/{projectUuid}/communities/{communityUuid}/spaces/{spaceUuid}/subspaces/{subSpaceUuid}/devices';
-  static const String deviceByUuid = '/device/{deviceUuid}';
-  static const String deviceFunctions = '/device/{deviceUuid}/functions';
-  static const String gatewayApi = '/device/gateway/{gatewayUuid}/devices';
+  static const String deviceByUuid = '/devices/{deviceUuid}';
+  static const String deviceFunctions = '/devices/{deviceUuid}/functions';
+  static const String gatewayApi = '/devices/gateway/{gatewayUuid}/devices';
  static const String deviceFunctionsStatus =
-      '/device/{deviceUuid}/functions/status';
+      '/devices/{deviceUuid}/functions/status';
  static const String powerClamp =
-      '/device/{powerClampUuid}/power-clamp/status';
+      '/devices/{deviceUuid}/functions/status';

  ///Device Permission Module
 //POST
@ -153,7 +153,8 @@ abstract class ApiEndpoints {
  static const String getScene = '/scene/tap-to-run/{sceneId}';
  static const String getIconScene = '/scene/icon';

-  static const String getUnitAutomation = '/projects/{projectId}/communities/{communityId}/spaces/{unitUuid}/automations';
+  static const String getUnitAutomation =
+      '/projects/{projectId}/communities/{communityId}/spaces/{unitUuid}/automations';

  static const String getAutomationDetails =
      '/projects/{projectId}/automations/{automationId}';
@ -161,7 +162,8 @@ abstract class ApiEndpoints {
  /// PUT
  static const String updateScene = '/scene/tap-to-run/{sceneId}';

-  static const String updateAutomation = '/projects/{projectId}/automations/{automationId}';
+  static const String updateAutomation =
+      '/projects/{projectId}/automations/{automationId}';

  static const String updateAutomationStatus =
      '/projects/{projectId}/automations/{automationId}';
@ -169,7 +171,8 @@ abstract class ApiEndpoints {
  /// DELETE
  static const String deleteScene = '/scene/tap-to-run/{sceneId}';

-  static const String deleteAutomation = '/projects/{projectId}/automations/{automationId}';
+  static const String deleteAutomation =
+      '/projects/{projectId}/automations/{automationId}';

  //////////////////////Door Lock //////////////////////
  //online
@ -213,18 +216,18 @@ abstract class ApiEndpoints {
  static const String changeSchedule = '/schedule/enable/{deviceUuid}';
  static const String deleteSchedule = '/schedule/{deviceUuid}/{scheduleId}';
  static const String reportLogs =
-      '/device/report-logs/{deviceUuid}?code={code}&startTime={startTime}&endTime={endTime}';
-  static const String controlBatch = '/device/control/batch';
-  static const String statusBatch = '/device/status/batch';
-  static const String deviceScene = '/device/{deviceUuid}/scenes';
+      '/devices/report-logs/{deviceUuid}?code={code}&startTime={startTime}&endTime={endTime}';
+  static const String controlBatch = '/devices/batch';
+  static const String statusBatch = '/devices/batch';
+  static const String deviceScene = '/devices/{deviceUuid}/scenes';

  static const String fourSceneByName =
-      '/device/{deviceUuid}/scenes?switchName={switchName}';
+      '/devices/{deviceUuid}/scenes?switchName={switchName}';

  static const String resetDevice = '/factory/reset/{deviceUuid}';
  static const String unAssignScenesDevice =
-      '/device/{deviceUuid}/scenes?switchName={switchName}';
-  static const String getDeviceLogs = '/device/report-logs/{uuid}?code={code}';
+      '/devices/{deviceUuid}/scenes?switchName={switchName}';
+  static const String getDeviceLogs = '/devices/report-logs/{uuid}?code={code}';
  static const String terms = '/terms';
  static const String policy = '/policy';
  static const String getPermission = '/permission/{roleUuid}';
--- a/lib/services/api/authentication_api.dart
+++ b/lib/services/api/authentication_api.dart
@ -25,11 +25,15 @@ class AuthenticationAPI {
    return response;
  }

-  static Future<bool> signUp({required SignUpModel model}) async {
+  static Future<bool> signUp({
+    required SignUpModel model,
+    required String accessToken,
+  }) async {
    final response = await HTTPService().post(
        path: ApiEndpoints.signUp,
        body: model.toJson(),
        showServerMessage: false,
+        accessToken: accessToken,
        expectedResponseModel: (json) => json['statusCode'] == 201);
    return response;
  }
@ -63,4 +67,20 @@ class AuthenticationAPI {
        expectedResponseModel: (json) => json['data']);
    return response;
  }
+
+  static Future<Map<String, dynamic>> fetchClientToken({
+    required String clientId,
+    required String clientSecret,
+  }) async {
+    final response = await HTTPService().post(
+      path: ApiEndpoints.clientLogin,
+      body: {
+        'clientId': clientId,
+        'clientSecret': clientSecret,
+      },
+      showServerMessage: false,
+      expectedResponseModel: (json) => json['data'],
+    );
+    return response;
+  }
 }
--- a/lib/services/api/devices_api.dart
+++ b/lib/services/api/devices_api.dart
@ -1,6 +1,5 @@
 import 'dart:async';
 import 'dart:convert';
-import 'dart:developer';
 import 'package:syncrow_app/features/devices/model/device_category_model.dart';
 import 'package:syncrow_app/features/devices/model/device_control_model.dart';
 import 'package:syncrow_app/features/devices/model/device_model.dart';
@ -8,7 +7,6 @@ import 'package:syncrow_app/features/devices/model/device_report_model.dart';
 import 'package:syncrow_app/features/devices/model/function_model.dart';
 import 'package:syncrow_app/services/api/api_links_endpoints.dart';
 import 'package:syncrow_app/services/api/http_service.dart';
-import 'package:syncrow_app/utils/constants/temp_const.dart';
 import '../../features/devices/model/create_temporary_password_model.dart';

 class DevicesAPI {
@ -39,7 +37,7 @@ class DevicesAPI {
        path: ApiEndpoints.deviceByUuid.replaceAll('{deviceUuid}', deviceId),
        body: {"deviceName": deviceName},
        expectedResponseModel: (json) {
-          return json;
+          return json['data'];
        },
      );
      return response;
@ -92,7 +90,7 @@ class DevicesAPI {
          .replaceAll('{deviceUuid}', deviceId),
      showServerMessage: false,
      expectedResponseModel: (json) {
-        return json;
+        return json['data'];
      },
    );
    return response;
@ -101,7 +99,7 @@ class DevicesAPI {
  static Future<Map<String, dynamic>> getPowerClampStatus(
      String deviceId) async {
    final response = await _httpService.get(
-      path: ApiEndpoints.powerClamp.replaceAll('{powerClampUuid}', deviceId),
+      path: ApiEndpoints.deviceFunctionsStatus.replaceAll('{powerClampUuid}', deviceId),
      showServerMessage: false,
      expectedResponseModel: (json) {
        return json;
@ -132,7 +130,7 @@ class DevicesAPI {
        path: ApiEndpoints.deviceFunctions.replaceAll('{deviceUuid}', deviceId),
        showServerMessage: false,
        expectedResponseModel: (json) {
-          final functions = FunctionModel.fromJson(json);
+          final functions = FunctionModel.fromJson(json['data']);
          return functions;
        });
    return response;
@ -188,7 +186,7 @@ class DevicesAPI {
        path: ApiEndpoints.deviceByUuid.replaceAll('{deviceUuid}', deviceId),
        showServerMessage: false,
        expectedResponseModel: (json) {
-          return json;
+          return json['data'];
        });
    return response;
  }
@ -264,7 +262,7 @@ class DevicesAPI {
        if (json == null || json.isEmpty || json == []) {
          return devices;
        }
-        for (var device in json['devices']) {
+        for (var device in json['data']['devices']) {
          devices.add(DeviceModel.fromJson(device));
        }
        return devices;
@ -491,7 +489,12 @@ class DevicesAPI {
  }) async {
    final response = await _httpService.post(
      path: ApiEndpoints.controlBatch,
-      body: {"devicesUuid": devicesUuid, "code": code, "value": value},
+      body: {
+        "devicesUuid": devicesUuid,
+        "code": code,
+        "value": value,
+        "operationType": 'COMMAND',
+      },
      showServerMessage: true,
      expectedResponseModel: (json) {
        return json;
--- a/lib/services/api/home_management_api.dart
+++ b/lib/services/api/home_management_api.dart
@ -34,7 +34,7 @@ class HomeManagementAPI {
        path: ApiEndpoints.devices.replaceAll("{projectUuid}", projectUuid),
        showServerMessage: false,
        expectedResponseModel: (json) {
-          json.forEach((value) {
+          json['data'].forEach((value) {
            list.add(DeviceModel.fromJson(value));
          });
        });
--- a/lib/services/api/http_service.dart
+++ b/lib/services/api/http_service.dart
@ -48,13 +48,26 @@ class HTTPService {
      Options? options,
      dynamic body,
      bool showServerMessage = true,
+      String? accessToken,
      required T Function(dynamic) expectedResponseModel}) async {
    try {
+      final authOptions = options ??
+          Options(
+            headers: accessToken != null
+                ? {
+                    'Authorization': 'Bearer $accessToken',
+                    'Content-Type': 'application/json',
+                  }
+                : {
+                    'Content-Type': 'application/json',
+                  },
+          );
+
      final response = await client.post(
        path,
        data: body,
        queryParameters: queryParameters,
-        options: options,
+        options: authOptions,
      );
      return expectedResponseModel(response.data);
    } catch (error) {
Author	SHA1	Message	Date
Faris Armoush	f2412aa867	refactor: extract device fetching logic into a separate method for improved readability	2025-04-16 16:10:36 +03:00
Faris Armoush	e4768c95aa	refactor: rename tab change events and states for consistency, and to match `bloc`s naming convention.	2025-04-16 16:07:30 +03:00
Faris Armoush	f25b4dbf6d	implement `SceneDevicesBodyTabBar` for improved tab management in `SceneDevicesBody`.	2025-04-16 15:57:21 +03:00
Faris Armoush	9eff9ab371	refactor: consolidate device loading and error handling in SceneDevicesBody and implement SceneDevicesList widget	2025-04-16 15:46:56 +03:00
Faris Armoush	a2e68d6194	fix: restore title in DefaultScaffold for SceneRoomsTabBarDevicesView	2025-04-16 15:25:23 +03:00
Faris Armoush	94c94b170f	fix: correct order of dispose method calls in SceneRoomsTabBarDevicesView	2025-04-16 15:24:47 +03:00
Faris Armoush	253cf15559	refactor: simplify SceneDevicesBody widget structure and improve loading indicator handling	2025-04-16 15:24:37 +03:00
Faris Armoush	443eea9421	SP-1397	2025-04-16 15:11:43 +03:00
hannathkadher	87a2c08f64	Merge pull request #82 from SyncrowIOT/SP-1416 added client token login	2025-04-16 12:35:31 +04:00
hannathkadher	dd66e7c747	added client token login	2025-04-16 12:02:19 +04:00
hannathkadher	7af61d2f65	added client id and secret	2025-04-16 11:34:24 +04:00
mohammadnemer1	f37eacb0ee	Merge pull request #81 from SyncrowIOT/feat/update-additional-device-endpoints updated endpoint for device	2025-04-14 15:11:36 +03:00
mohammad	c7dcc297aa	fix conflict	2025-04-14 15:11:02 +03:00
mohammad	aefe5ad081	Merge branch 'dev' into feat/update-additional-device-endpoints	2025-04-14 15:10:42 +03:00
mohammadnemer1	8d9af8519d	Merge pull request #78 from SyncrowIOT/SP-1246 updated device endpoint	2025-04-14 14:58:09 +03:00
hannathkadher	b1d52937c9	Merge branch 'dev' of https://github.com/SyncrowIOT/syncrow-app into feat/update-additional-device-endpoints	2025-04-14 10:31:09 +04:00
hannathkadher	5fedf37421	updated endpoint for device	2025-04-14 10:30:48 +04:00
hannathkadher	4c8f2c72df	Merge branch 'dev' of https://github.com/SyncrowIOT/syncrow-app into SP-1246	2025-04-03 10:47:27 +04:00
hannathkadher	fb867e5df3	updated device endpoint	2025-04-03 10:47:09 +04:00