Remove useless TODOs

Author: TimoPtr

app/src/main/kotlin/io/homeassistant/companion/android/onboarding/OnboardingNavigation.kt

@@ -422,7 +422,6 @@ internal fun NavGraphBuilder.wearOnboarding(
     }
 
     navigation<WearOnboardingRoute>(startDestination = startRoute) {
-        // TODO discovery should be able to add existing system
         commonScreens(navController = navController, wearNameToOnboard = wearNameToOnboard)
         nameYourWearDeviceScreen(
             onBackClick = navController::popBackStack,
@@ -451,6 +450,5 @@ internal fun NavGraphBuilder.wearOnboarding(
             },
             onNext = onOnboardingDone,
         )
-        // TODO: Consider making auth_code a value class to prevent string parameter mismatches
     }
 }

app/src/main/kotlin/io/homeassistant/companion/android/onboarding/locationsharing/LocationSharingViewModel.kt

@@ -8,6 +8,7 @@ import androidx.navigation.toRoute
 import dagger.hilt.android.lifecycle.HiltViewModel
 import io.homeassistant.companion.android.database.sensor.SensorDao
 import io.homeassistant.companion.android.onboarding.locationsharing.navigation.LocationSharingRoute
+import io.homeassistant.companion.android.sensors.LocationSensorManager
 import javax.inject.Inject
 import kotlinx.coroutines.launch
 import timber.log.Timber
@@ -29,10 +30,9 @@ internal class LocationSharingViewModel @VisibleForTesting constructor(
             try {
                 sensorDao.setSensorsEnabled(
                     sensorIds = listOf(
-                        // TODO add sensor ID from `LocationSensorManager` instead of string
-                        "location_background",
-                        "zone_background",
-                        "accurate_location",
+                        LocationSensorManager.backgroundLocation.id,
+                        LocationSensorManager.zoneLocation.id,
+                        LocationSensorManager.singleAccurateLocation.id,
                     ),
                     serverId = serverId,
                     enabled = enabled,

app/src/main/kotlin/io/homeassistant/companion/android/onboarding/manualserver/ManualServerScreen.kt

@@ -194,7 +194,6 @@ private fun ServerUrlTextField(
             if (isError) {
                 Text(
                     text = stringResource(commonR.string.manual_server_wrong_url),
-                    // TODO probably wrong style and color/token
                     style = HATextStyle.BodyMedium.copy(color = LocalHAColorScheme.current.colorBorderDangerNormal),
                 )
             }

app/src/main/kotlin/io/homeassistant/companion/android/onboarding/serverdiscovery/ServerDiscoveryScreen.kt

@@ -191,8 +191,7 @@ private fun OneServerFound(
                     imageVector = Icons.Default.Storage,
                     contentDescription = null,
                     modifier = Modifier
-                        .size(ICON_SIZE), // TODO double check the size of the icon within the modal
-                    // TODO change the color with proper token
+                        .size(ICON_SIZE),
                     tint = LocalHAColorScheme.current.colorFillPrimaryLoudResting,
                 )
                 Text(
@@ -384,7 +383,7 @@ private fun AnimatedIcon() {
                 .size(80.dp)
                 .scale(pulse)
                 .align(Alignment.Center)
-                .background(HABrandColors.Blue, CircleShape), // TODO we might want to use a semantic token?
+                .background(HABrandColors.Blue, CircleShape),
         ) {
             Icon(
                 imageVector = ImageVector.vectorResource(commonR.drawable.ic_stat_ic_notification_blue),