Closed thehale closed 4 months ago
Tagging @Razeeman for a review since I'm not sure if GitHub sends notifications for Draft PRs.
Hello! Thank you very much for the help. Looks cool! Looking through the code.
Also if it is possible could you update thehale:wearos
branch to the recent dev? Commit 9210757 should already be there and shoud not be shown here in the changes. To avoid git history pollution.
If you don't mind I would merge and start working on preparing the release. Other changes can be made in separate PRs if there would be any.
Your PR motivated me to buy a watch and it would super cool to see it in action. Thank you very much for implementing it.
β Simple Time Tracker for WearOS
Adds a WearOS companion app to enable changing active activities from one's wrist.
Features:
closes #60
https://github.com/Razeeman/Android-SimpleTimeTracker/assets/47901316/d2a9735d-880b-484d-8f57-419b4770c1e6
π Important Implementation Notes for Review
π² Phone <-> β Wear Communication Protocol
Reliable communication between the phone and the watch can be tricky. Instead of puzzling over the code directly, I recommend first reading through the documentation of the protocol.
π₯This PR drops support for Android versions < 8.0
WearOS requires at least a
minSDK
of 23 (Android 6.0). The libraries I used to implement the app require at leastminSDK
of 26 (Android 8.0).This is a significant change from Simple Time Tracker's current support for devices as old as Android 5.0. We'll have to discuss the amount of backward compatibility we want to include after incorporating a WearOS app.
According to Android Studio, here are the percent of active Android devices running at each API version as of today.
βοΈ MPL-2.0 Licensing
I would prefer to contribute my code under the terms of the MPL-2.0 license, which is fully compatible with the GPL-3.0 used in this repository.