From 76dc504f7e7476e8920c2f9d18314c5ec519c2ee Mon Sep 17 00:00:00 2001 From: Abhijeet Prasad Bodas Date: Thu, 8 Jul 2021 10:24:27 +0530 Subject: [PATCH] push_notifications: Remove outdated comments. We solely rely on the logic here for filtering with the `read` flag since 10dd5f784bd4c4399f280e6faabf6eb36792957c. --- zerver/lib/push_notifications.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/zerver/lib/push_notifications.py b/zerver/lib/push_notifications.py index 511f376d36..b06d2a2b4a 100644 --- a/zerver/lib/push_notifications.py +++ b/zerver/lib/push_notifications.py @@ -912,10 +912,6 @@ def handle_push_notification(user_profile_id: int, missed_message: Dict[str, Any if user_message is not None: # If the user has read the message already, don't push-notify. - # - # TODO: It feels like this is already handled when things are - # put in the queue; maybe we should centralize this logic with - # the `zerver/tornado/event_queue.py` logic? if user_message.flags.read or user_message.flags.active_mobile_push_notification: return