Merge pull request #1813 from bluesky-social/eric/app-903-extract-logger-into-singleton

Add new logger

1 files changed, 9 insertions, 10 deletions

diff --git a/src/lib/notifications/notifications.ts b/src/lib/notifications/notifications.ts
index dfc9a42b1..01b0ba935 100644
--- a/src/lib/notifications/notifications.ts
+++ b/src/lib/notifications/notifications.ts
@@ -30,18 +30,18 @@ export function init(store: RootStoreModel) {
           appId: 'xyz.blueskyweb.app',
         })
         store.log.debug('Notifications: Sent push token (init)', {
-          type: token.type,
+          tokenType: token.type,
           token: token.data,
         })
       } catch (error) {
-        store.log.error('Notifications: Failed to set push token', error)
+        store.log.error('Notifications: Failed to set push token', {error})
       }
     }
 
     // listens for new changes to the push token
     // In rare situations, a push token may be changed by the push notification service while the app is running. When a token is rolled, the old one becomes invalid and sending notifications to it will fail. A push token listener will let you handle this situation gracefully by registering the new token with your backend right away.
     Notifications.addPushTokenListener(async ({data: t, type}) => {
-      store.log.debug('Notifications: Push token changed', {t, type})
+      store.log.debug('Notifications: Push token changed', {t, tokenType: type})
       if (t) {
         try {
           await store.agent.api.app.bsky.notification.registerPush({
@@ -51,11 +51,11 @@ export function init(store: RootStoreModel) {
             appId: 'xyz.blueskyweb.app',
           })
           store.log.debug('Notifications: Sent push token (event)', {
-            type,
+            tokenType: type,
             token: t,
           })
         } catch (error) {
-          store.log.error('Notifications: Failed to set push token', error)
+          store.log.error('Notifications: Failed to set push token', {error})
         }
       }
     })
@@ -63,7 +63,7 @@ export function init(store: RootStoreModel) {
 
   // handle notifications that are received, both in the foreground or background
   Notifications.addNotificationReceivedListener(event => {
-    store.log.debug('Notifications: received', event)
+    store.log.debug('Notifications: received', {event})
     if (event.request.trigger.type === 'push') {
       // refresh notifications in the background
       store.me.notifications.syncQueue()
@@ -84,10 +84,9 @@ export function init(store: RootStoreModel) {
   // handle notifications that are tapped on
   const sub = Notifications.addNotificationResponseReceivedListener(
     response => {
-      store.log.debug(
-        'Notifications: response received',
-        response.actionIdentifier,
-      )
+      store.log.debug('Notifications: response received', {
+        actionIdentifier: response.actionIdentifier,
+      })
       if (
         response.actionIdentifier === Notifications.DEFAULT_ACTION_IDENTIFIER
       ) {