-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: Separate start and init tracking
We used to rely on a single `startTracking` method that was both initializating the tracking, and started it. Furthermore, the event handlers were made outside of this function. This seems to be an anti-pattern, and we suspect that it was causing tracking failures: we noticed several cases of missing motion events, which might be the result of a wrong init. Now, the init is done at each app start, and the tracking is actually started when we detect the state is not enabled, just like before.
- Loading branch information
1 parent
4a0a7d1
commit 7c913db
Showing
3 changed files
with
51 additions
and
22 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters