Skip to content

Add start workers in LaunchActivity #6720

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
TimoPtr merged 5 commits into from
Apr 23, 2026
Merged

Add start workers in LaunchActivity #6720

Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
19a7b7f
Add start workers in FrontendScreen
TimoPtr Apr 16, 2026
0ea972e
Move the logic into the LaunchActivtiy
TimoPtr Apr 20, 2026
6d9e728
Apply PR feedback
TimoPtr Apr 20, 2026
fbb8e95
Apply suggestions
TimoPtr Apr 22, 2026
6af7901
Add unit test to verify the worker and update are properly invoked
TimoPtr Apr 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 26 additions & 1 deletion app/src/main/kotlin/io/homeassistant/companion/android/launch/LaunchActivity.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,25 +18,37 @@ import androidx.compose.ui.res.stringResource
import androidx.core.content.IntentCompat
import androidx.core.splashscreen.SplashScreen.Companion.installSplashScreen
import androidx.lifecycle.compose.collectAsStateWithLifecycle
import androidx.lifecycle.lifecycleScope
import androidx.navigation.NavController
import androidx.navigation.compose.rememberNavController
import dagger.hilt.android.AndroidEntryPoint
import dagger.hilt.android.lifecycle.withCreationCallback
import io.homeassistant.companion.android.common.R as commonR
import io.homeassistant.companion.android.common.compose.theme.HATheme
import io.homeassistant.companion.android.sensors.SensorReceiver
import io.homeassistant.companion.android.sensors.SensorWorker
import io.homeassistant.companion.android.util.PLAY_SERVICES_FLAVOR_DOC_URL
import io.homeassistant.companion.android.util.PlayServicesAvailability
import io.homeassistant.companion.android.util.compose.HAApp
import io.homeassistant.companion.android.util.compose.navigateToUri
import io.homeassistant.companion.android.util.enableEdgeToEdgeCompat
import io.homeassistant.companion.android.websocket.WebsocketManager
import javax.inject.Inject
import kotlinx.coroutines.launch
import kotlinx.parcelize.Parcelize

private const val DEEP_LINK_KEY = "deep_link_key"

/**
* Main entry point of the application, it is mostly responsible to hold the whole navigation of the application.
* Main entry point of the application, responsible for holding the whole navigation graph
* and managing background workers tied to the Activity lifecycle.
*
* It also handles the splash screen display based on a condition exposed by the [LaunchViewModel].
*
* On resume, starts periodic sensor collection via [SensorWorker] and the persistent
* WebSocket connection via [WebsocketManager].
Comment thread
TimoPtr marked this conversation as resolved.
Outdated
* On pause, triggers an immediate sensor update via [SensorReceiver] so the server
* has fresh data before the app goes to the background.
*/
@AndroidEntryPoint
class LaunchActivity : AppCompatActivity() {
Expand Down Expand Up @@ -94,6 +106,19 @@ class LaunchActivity : AppCompatActivity() {
},
)

override fun onResume() {
Comment thread
jpelgrom marked this conversation as resolved.
Outdated
super.onResume()
SensorWorker.start(this)
lifecycleScope.launch {
WebsocketManager.start(this@LaunchActivity)
}
}

override fun onPause() {
super.onPause()
SensorReceiver.updateAllSensors(this)
}
Comment thread
TimoPtr marked this conversation as resolved.
Outdated

override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState)
val splashScreen = installSplashScreen()
Expand Down
Loading