Merge pull request #63 from Qabel/m/fix-fcm

Remove message from body due to length limits
Qabel · Aug 8, 2016 · 2998ac6 · 2998ac6
2 parents 78ce350 + 122f3f4
commit 2998ac6
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/drop_service/notify.py b/drop_service/notify.py
@@ -41,7 +41,7 @@ def notify(self, drop):
     def _notify(self, drop):
         data = {
             'drop-id': drop.drop_id,
-            'message': base64.b64encode(drop.message).decode(),
+        #    'message': base64.b64encode(drop.message).decode(),
         }
         # Alphabet of topics: [a-zA-Z0-9-_.~%]
         # Alphabet of drop IDs: [a-zA-Z0-9-_]

diff --git a/requirements.txt b/requirements.txt
@@ -10,4 +10,4 @@ djangorestframework==3.3.3
 pytest-django==2.9.1
 invoke==0.13.0
 pprintpp==0.2.3
-pyfcm==1.0.0
+pyfcm==1.0.2
diff --git a/tests/test_notify.py b/tests/test_notify.py
@@ -16,7 +16,6 @@ def test_fcm_basic(drop_messages):
         fcm.notify(drop1)
 
     data_message = {
-        'message': 'SGVsbG8gV29ybGQ=',
         'drop-id': 'abcdefghijklmnopqrstuvwxyzabcdefghijklmnopo'
     }
     api.notify_topic_subscribers.assert_called_once_with(topic_name=drop1.drop_id, data_message=data_message)