misc todo's

Author: Salakar

lib/modules/messaging.js

@@ -16,22 +16,17 @@ export default class Messaging extends Base {
   /*
    * WEB API
    */
+  // TODO move to new event emitter logic
   onMessage(callback) {
     this.log.info('Setting up onMessage callback');
     const sub = this._on('FirestackReceiveNotification', callback, FirestackMessagingEvt);
     return promisify(() => sub, FirestackMessaging)(sub);
   }
 
-  // android & ios api dfgsdfs
-  onMessageReceived(...args) {
-    return this.onMessage(...args);
-  }
-
-  // there is no 'off' for this on api's but it's needed here for react
-  // so followed the general 'off' / 'on' naming convention
+  // TODO this is wrong - also there is no 'off' onMessage should return the unsubscribe function
   offMessage() {
     this.log.info('Unlistening from onMessage (offMessage)');
-    this._off('FirestackRefreshToken');
+    this._off('FirestackReceiveNotification');
   }
 
   offMessageReceived(...args) {