Skip to content

Commit

Permalink
push_notifications: Remove outdated comments.
Browse files Browse the repository at this point in the history
We solely rely on the logic here for filtering with the `read` flag
since 10dd5f7.
  • Loading branch information
abhijeetbodas2001 authored and timabbott committed Jul 8, 2021
1 parent af47fa7 commit 76dc504
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions zerver/lib/push_notifications.py
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down

0 comments on commit 76dc504

Please sign in to comment.