Merge branch 'main' into 63-reduce-the-time-between-initial-learning-event-and-the-1st-review

Author: jo-elimu

.github/pull_request_template.md

@@ -0,0 +1,23 @@
+### Issue Number
+<!-- Which issue does this PR address? E.g. "Resolves #123" -->
+* Resolves #
+
+### Purpose
+<!-- What is the purpose of this PR? Why is it needed? -->
+* 
+
+### Technical Details
+<!-- Are there any key aspects of the implementation to highlight? -->
+* 
+
+### Testing Instructions
+<!-- How can the reviewer verify the functionality or fix introduced by this PR? Please provide steps. -->
+* 
+
+### Regression Testing
+<!-- Have you verified that previously developed software still performs as expected? -->
+- [ ] I have tested that the changes in this pull request didn't break any existing functionality
+
+### Screenshots
+<!-- If this PR affects the UI, please include before/after screenshots demonstrating the change(s). -->
+*

app/src/main/java/ai/elimu/kukariri/service/ScreenOnService.kt

@@ -8,6 +8,9 @@ import android.os.IBinder
 import android.util.Log
 
 class ScreenOnService : Service() {
+
+    private lateinit var screenOnReceiver: ScreenOnReceiver
+
     override fun onBind(intent: Intent): IBinder? {
         return null
     }
@@ -16,9 +19,18 @@ class ScreenOnService : Service() {
         Log.i(javaClass.name, "onStartCommand")
 
         // Register receiver for detecting when the screen is turned on
-        val screenOnReceiver = ScreenOnReceiver()
+        screenOnReceiver = ScreenOnReceiver()
         registerReceiver(screenOnReceiver, IntentFilter(Intent.ACTION_SCREEN_ON))
 
         return super.onStartCommand(intent, flags, startId)
     }
+
+    override fun onDestroy() {
+        super.onDestroy()
+        try {
+            unregisterReceiver(screenOnReceiver)
+        } catch (e: Exception) {
+            Log.e("ScreenOnService", "Error unregistering receiver", e)
+        }
+    }
 }