Only send the ping if it actually changed

ttwefff · May 24, 2017 · c388541 · c388541
1 parent 5ae8aaf
commit c388541
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/app/src/lib/stats/stats-store.ts b/app/src/lib/stats/stats-store.ts
@@ -232,11 +232,17 @@ export class StatsStore {
 
   /** Set whether the user has opted out of stats reporting. */
   public setOptOut(optOut: boolean): Promise<void> {
+    const changed = this.optOut !== optOut
+
     this.optOut = optOut
 
     localStorage.setItem(StatsOptOutKey, optOut ? '1' : '0')
 
-    return this.sendOptInStatusPing(!optOut)
+    if (changed) {
+      return this.sendOptInStatusPing(!optOut)
+    } else {
+      return Promise.resolve()
+    }
   }
 
   /** Has the user opted out of stats reporting? */