[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-10-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #23 from Andre Klapper aklap...@wikimedia.org --- Is Nischay still working on this, or should the assignee be reset to default? -- You are receiving this mail because: You are on the CC list for the bug.

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-09-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #22 from sp...@wikimedia.org --- Prioritization and scheduling of this bug is tracked on Mingle card https://mingle.corp.wikimedia.org/projects/flow/cards/198 -- You are receiving this mail because: You are on the CC list for the

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-07-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #21 from Nemo federicol...@tiscali.it --- (In reply to comment #20) Possibly related to the very old Bug 2939 ('You've got new message is displayed even after a revert') Well, it's related in the sense that Echo probably makes

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-07-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 Quiddity pandiculat...@gmail.com changed: What|Removed |Added CC|

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-05-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 Nemo federicol...@tiscali.it changed: What|Removed |Added See Also|

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #19 from Nemo federicol...@tiscali.it --- (In reply to comment #16) @Nemo: In the case that JarJar vandalizes Kirk's userpage, and BuckRogers reverts JarJar, should Kirk receive one notification about the rollback or none at

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #15 from Ryan Kaldari rkald...@wikimedia.org --- The fast and easy way is just to check request variables: wpUndidRevision exists or action == rollback before creating a talk page notification, if the author of the rollback revision

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #16 from Ryan Kaldari rkald...@wikimedia.org --- Actually I guess this bug might be about both cases, but I'm not sure from the bug's initial description. @Nemo: In the case that JarJar vandalizes Kirk's userpage, and BuckRogers

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #17 from Ryan Kaldari rkald...@wikimedia.org --- Either way, we wouldn't want to suppress the notification about the rollback unless we also suppressed the notification about the vandal edit that was rolled back, so if there is no

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #18 from Luke Welling lwell...@wikimedia.org --- Attempting to address this would take significant work and depending on the exact timing and user settings its behaviour could be very mysterious to users. With a similar degree of

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 Nischay Nahata nischay...@gmail.com changed: What|Removed |Added See Also|

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #10 from Ryan Kaldari rkald...@wikimedia.org --- Oh yeah, looks like you're right. We may want to add revision as it's own column in that case. -- You are receiving this mail because: You are on the CC list for the bug.

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #11 from Nischay Nahata nischay...@gmail.com --- (In reply to comment #10) Oh yeah, looks like you're right. We may want to add revision as it's own column in that case. Does the design team have a say on this? or can we just add

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #12 from Ryan Kaldari rkald...@wikimedia.org --- We can just add the column, but I'd like to make sure Benny thinks it's a good idea since he's handling the back-end components. -- You are receiving this mail because: You are on

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #13 from Terry Chay tc...@wikimedia.org --- Adding Luke since this will affect Redis mailbox. -- You are receiving this mail because: You are on the CC list for the bug. ___ Wikibugs-l

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-15 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #14 from bs...@wikimedia.org --- It wouldn't help if we add a revision column + extra index to event table. Upon each event is created, web + email notifications would be either created immediately or a job would be scheduled right

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #9 from Nischay Nahata nischay...@gmail.com --- (In reply to comment #8) take the ID of the rev that was rolled back and look for any echo events in the database related to this rev and delete them. Just be sure to do this before

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #5 from Nischay Nahata nischay...@gmail.com --- I think some users may be against this. You don't want to see it if its spam, what if its not? Hiding details like this is not always appreciated. Also how should this be done? when

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #6 from Terry Chay tc...@wikimedia.org --- I think some users may be against this. You don't want to see it if its spam, what if its not? The goal is not to suppress the notification, it's that one event is generating two

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #7 from Nischay Nahata nischay...@gmail.com --- (In reply to comment #6) Ask kaldari on #wikimedia-ee for information on merging multiple redundant notifications in Echo. There is a system in place for doing this, it just needs

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #8 from Ryan Kaldari rkald...@wikimedia.org --- Probably the way this would be implemented is in EchoHooks::onRollbackComplete take the ID of the rev that was rolled back and look for any echo events in the database related to this

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 Nemo federicol...@tiscali.it changed: What|Removed |Added See Also|

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 Ryan Kaldari rkald...@wikimedia.org changed: What|Removed |Added Assignee|wikibugs-l@lists.wikimedia.

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 Terry Chay tc...@wikimedia.org changed: What|Removed |Added CC||tc...@wikimedia.org

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-03 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #3 from Nemo federicol...@tiscali.it --- (In reply to comment #2) Right now we are just mirroring the behavior of the existing talk page notification system. I'm not sure, what notification system are you talking about? The new

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-04-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #2 from Ryan Kaldari rkald...@wikimedia.org --- Right now we are just mirroring the behavior of the existing talk page notification system. While I agree it would probably be better to suppress notifications about rolledback edits,

[Bug 46689] Rollback must suppress notifications for rollbacked edit

2013-03-29 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=46689 --- Comment #1 from Nemo federicol...@tiscali.it --- To be precise, the example in screenshot was not a rollback but a simple revert. Rollback *must* suppress notifications, while other revert systems may need some more work. I suppose,