From ecf23b6b2edb67466c1f7fdcd3c5890de3a73ba3 Mon Sep 17 00:00:00 2001
From: Daniel Gasienica <daniel@signal.org>
Date: Wed, 2 May 2018 19:58:23 -0400
Subject: [PATCH] Disable audio notifications on Windows 7 and lower

---
 ts/test/types/Settings_test.ts | 15 +++++++++++++++
 ts/types/Settings.ts           |  4 +++-
 2 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/ts/test/types/Settings_test.ts b/ts/test/types/Settings_test.ts
index 306e0ef99..ebd7fd10d 100644
--- a/ts/test/types/Settings_test.ts
+++ b/ts/test/types/Settings_test.ts
@@ -23,6 +23,21 @@ describe('Settings', () => {
     });
 
     context('on Windows', () => {
+      context('version 7', () => {
+        beforeEach(() => {
+          sandbox.stub(process, 'platform').value('win32');
+          sandbox.stub(os, 'release').returns('7.0.0');
+        });
+
+        afterEach(() => {
+          sandbox.restore();
+        });
+
+        it('should return false', () => {
+          assert.isFalse(Settings.isAudioNotificationSupported());
+        });
+      });
+
       context('version 8+', () => {
         beforeEach(() => {
           sandbox.stub(process, 'platform').value('win32');
diff --git a/ts/types/Settings.ts b/ts/types/Settings.ts
index 521106230..08eea3106 100644
--- a/ts/types/Settings.ts
+++ b/ts/types/Settings.ts
@@ -1,4 +1,6 @@
 import * as OS from '../OS';
 
+const MIN_WINDOWS_VERSION = '8.0.0';
+
 export const isAudioNotificationSupported = () =>
-  OS.isWindows() || OS.isMacOS();
+  OS.isWindows(MIN_WINDOWS_VERSION) || OS.isMacOS();