如果关注然后回复中又@了会提醒多次 #92

Closed
cybersingle opened this Issue Sep 5, 2012 · 1 comment

3 participants

@cybersingle
# 给关注者发通知
topic.follower_ids.each do |uid|

下面应该屏蔽掉这种情况
# 排除@中已经发送过通知的人
next if reply.mentioned_user_ids.include?(uid)

@huacnlee huacnlee was assigned Nov 13, 2012
@ichord

这个 bug 已经修复了吧? 可以关掉?

@huacnlee huacnlee added a commit that closed this issue Jun 14, 2013
@huacnlee huacnlee Fix #92, 由于 @ 的通知发出于 follower 的通知不是在同一个生命周期(Rails 和 Sidekiq),所以,之前 @Rei
… 设计的 notified_user_ids 临时存放发送过的用户编号会不起作用.
6f34376
@huacnlee huacnlee closed this in 6f34376 Jun 14, 2013
@testerhome testerhome pushed a commit that referenced this issue Dec 30, 2013
@huacnlee huacnlee Fix #92, 由于 @ 的通知发出于 follower 的通知不是在同一个生命周期(Rails 和 Sidekiq),所以,之前 @Rei
… 设计的 notified_user_ids 临时存放发送过的用户编号会不起作用.
5eaead4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment