home-assistant / home-assistant/android

Review coroutine scopes in SensorManager

Open
#5,585 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Kotlin
Stars
3.9k
Forks
1.1k
Avg merge
1d 12h
Merged PRs (30d)
81

Description

On the effort of making sure that DB calls are not made on the main thread on this PR https://github.com/home-assistant/android/pull/5517 we've started discussing about the general scope usage in it. It exposes a Main scope and everything is launch into it, we should double check if it still make sense and if we can't put some work into an IO scope.

Alongside with this we need to double check the usage of `Job` vs `SupervisedJob`.

For more context: https://github.com/home-assistant/android/pull/5517#discussion_r2223406457

Any attempt to work on this should result into cleaning the TODO in the code base associated with this issue.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.