Update comments
This commit is contained in:
parent
e440e28456
commit
573712da6b
|
@ -190,12 +190,13 @@ class BulkPushRuleEvaluator(object):
|
||||||
if matches:
|
if matches:
|
||||||
actions = [x for x in rule['actions'] if x != 'dont_notify']
|
actions = [x for x in rule['actions'] if x != 'dont_notify']
|
||||||
if actions and 'notify' in actions:
|
if actions and 'notify' in actions:
|
||||||
|
# Push rules say we should notify the user of this event
|
||||||
actions_by_user[uid] = actions
|
actions_by_user[uid] = actions
|
||||||
break
|
break
|
||||||
|
|
||||||
# Push rules say we should notify the user of this event,
|
# Mark in the DB staging area the push actions for users who should be
|
||||||
# so we mark it in the DB in the staging area. (This
|
# notified for this event. (This will then get handled when we persist
|
||||||
# will then get handled when we persist the event)
|
# the event)
|
||||||
yield self.store.add_push_actions_to_staging(
|
yield self.store.add_push_actions_to_staging(
|
||||||
event.event_id, actions_by_user,
|
event.event_id, actions_by_user,
|
||||||
)
|
)
|
||||||
|
|
Loading…
Reference in New Issue