From 99eec243a69da96f65df22156d66f49de3d5dff0 Mon Sep 17 00:00:00 2001 From: Daniel Silva Date: Wed, 10 Apr 2024 09:45:46 +0200 Subject: [PATCH] adding log lines --- .../subscribeToReportCommentPushNotifications.ts | 2 +- src/libs/actions/User.ts | 7 ++++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/libs/Notification/PushNotification/subscribeToReportCommentPushNotifications.ts b/src/libs/Notification/PushNotification/subscribeToReportCommentPushNotifications.ts index 8d64f163217a..36051fa35c56 100644 --- a/src/libs/Notification/PushNotification/subscribeToReportCommentPushNotifications.ts +++ b/src/libs/Notification/PushNotification/subscribeToReportCommentPushNotifications.ts @@ -32,7 +32,7 @@ Onyx.connect({ export default function subscribeToReportCommentPushNotifications() { PushNotification.onReceived(PushNotification.TYPE.REPORT_COMMENT, ({reportID, reportActionID, onyxData, lastUpdateID, previousUpdateID}) => { if (!ActiveClientManager.isClientTheLeader()) { - Log.info(`[PushNotification] received report comment notification, but ignoring it since this is not the active client`); + Log.info('[PushNotification] received report comment notification, but ignoring it since this is not the active client'); return; } Log.info(`[PushNotification] received report comment notification in the ${Visibility.isVisible() ? 'foreground' : 'background'}`, false, {reportID, reportActionID}); diff --git a/src/libs/actions/User.ts b/src/libs/actions/User.ts index e552e818c56e..4740c2dda22b 100644 --- a/src/libs/actions/User.ts +++ b/src/libs/actions/User.ts @@ -586,6 +586,7 @@ function subscribeToUserEvents() { PusherUtils.subscribeToPrivateUserChannelEvent(Pusher.TYPE.MULTIPLE_EVENTS, currentUserAccountID.toString(), (pushJSON) => { // If this is not the main client, we shouldn't process any data received from pusher. if (!ActiveClientManager.isClientTheLeader()) { + Log.info('[Pusher] Received updates, but ignoring it since this is not the active client'); return; } // The data for the update is an object, containing updateIDs from the server and an array of onyx updates (this array is the same format as the original format above) @@ -606,7 +607,11 @@ function subscribeToUserEvents() { return SequentialQueue.getCurrentRequest().then(() => { // If we don't have the currentUserAccountID (user is logged out) or this is not the // main client we don't want to update Onyx with data from Pusher - if (currentUserAccountID === -1 || !ActiveClientManager.isClientTheLeader()) { + if (currentUserAccountID === -1) { + return; + } + if(!ActiveClientManager.isClientTheLeader()) { + Log.info('[Pusher] Received updates, but ignoring it since this is not the active client'); return; }