fix: disabling battery savings takes some extra time in certain phones

GeoTecINIT · Apr 5, 2022 · 3e2265b · 3e2265b
1 parent 72dac57
commit 3e2265b
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/publish/publish.sh b/publish/publish.sh
diff --git a/src/internal/tasks/schedulers/time-based/android/alarms/perm-request-common.ts b/src/internal/tasks/schedulers/time-based/android/alarms/perm-request-common.ts
@@ -17,3 +17,7 @@ export function waitForActivityResume(): Promise<void> {
     );
   });
 }
+
+export function fireInMs(ms: number): Promise<void> {
+  return new Promise<void>((resolve) => setTimeout(resolve, ms));
+}
diff --git a/src/internal/tasks/schedulers/time-based/android/alarms/power-savings-manager.android.ts b/src/internal/tasks/schedulers/time-based/android/alarms/power-savings-manager.android.ts
@@ -1,7 +1,9 @@
 import { Application, Utils } from "@nativescript/core";
 import { createSavingsDeactivationIntent } from "../intents.android";
 import { Logger, getLogger } from "../../../../../utils/logger";
-import { waitForActivityResume } from "./perm-request-common";
+import { fireInMs, waitForActivityResume } from "./perm-request-common";
+
+const MS_TO_UPDATE = 1000;
 
 export class PowerSavingsManager {
   private logger: Logger;
@@ -48,6 +50,9 @@ export class PowerSavingsManager {
 
     await activityResume;
 
+    // In certain phones the savings update is not immediate
+    await fireInMs(MS_TO_UPDATE);
+
     if (!this.areDisabled()) {
       throw new Error(
         "Disabling battery optimizations is required for the app to work as expected!"
-Original file line number
+Diff line change
@@ Expand Up / @@ -17,3 +17,7 @@ export function waitForActivityResume(): Promise<void> { @@
         );
       });
     }
+    export function fireInMs(ms: number): Promise<void> {
+      return new Promise<void>((resolve) => setTimeout(resolve, ms));
+    }