Merge pull request #171 from appfeel/fix/silent-high-priority-gcm

Support silent high priority notifications for GCM
appfeel · Jan 6, 2022 · 72c3364 · 72c3364
2 parents b6294c5 + b31c0b9
commit 72c3364
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/sendGCM.js b/src/sendGCM.js
@@ -134,7 +134,7 @@ const sendGCM = (regIds, data, settings) => {
   const message = new gcm.Message({
     // See https://developers.google.com/cloud-messaging/http-server-ref#table5
     collapseKey: data.collapseKey,
-    priority: data.priority === 'normal' || data.silent ? 'normal' : 'high',
+    priority: data.priority === 'normal' ? 'normal' : 'high',
     contentAvailable: data.silent ? true : data.contentAvailable || false,
     delayWhileIdle: data.delayWhileIdle || false,
     timeToLive: extractTimeToLive(data),

diff --git a/test/send/sendGCM.js b/test/send/sendGCM.js
@@ -708,7 +708,7 @@ describe('push-notifications-gcm', () => {
           expect(retries).to.be.a('number');
           expect(message).to.be.instanceOf(gcm.Message);
           expect(message.params.notification).to.be.undefined();
-          expect(message.params.priority).to.equal('normal');
+          expect(message.params.priority).to.equal('high');
           expect(message.params.contentAvailable).to.be.true();
           expect(message.params.data.testKey).to.eql('testValue');
           expect(message.params.data.title).to.be.undefined();