Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content

Introduce WeatherUpdater instead of workers

Jonathan Klee requested to merge 6236-remove-workers into release-1.6-rc

Description

Introduce WeatherUpdater instead of workers.

Technical details

Launching a Worker seems to trigger a PACKAGE_UPDATED intent associated to foundation.e.blisslauncher (which is really weird). BlissLauncher detects it and then update the app icons. This is what causes the closing of the window.

I created a WeatherUpdater singleton to replace the 2 weather workers.

Tests

https://gitlab.e.foundation/e/os/BlissLauncher/-/quality/test_cases/191

Issues

e/backlog#6236 (closed)

10 commandments of code reviews

👪 https://gitlab.e.foundation/internal/wiki/-/wikis/development/code-review

Edited by Jonathan Klee

Merge request reports

Loading