Page MenuHomePhabricator

Don't re-mention users for comment edits
ClosedPublic

Authored by epriestley on Jun 13 2016, 6:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:04 AM
Unknown Object (File)
Tue, Apr 9, 8:30 PM
Unknown Object (File)
Mon, Apr 8, 1:33 AM
Unknown Object (File)
Mon, Apr 1, 2:31 PM
Unknown Object (File)
Fri, Mar 29, 7:44 PM
Unknown Object (File)
Fri, Mar 29, 5:43 PM
Unknown Object (File)
Fri, Mar 29, 5:43 PM
Unknown Object (File)
Fri, Mar 29, 5:43 PM
Subscribers
None

Details

Summary

Ref T11035. This only fixes half of the issue: comment editing has been fixed, but normal transactions which edit things like descriptions haven't yet.

The normal edits aren't fixed because the "oldValues" are populated too late. The code should start working once they get populated sooner, but I don't want to jump the gun on that since it'll probably have some spooky effects. I have some other transaction changes coming down the pipe which should provide a better context for testing "oldValue" population order.

Test Plan
  • Mentioned @dog in a comment.
  • Removed @dog as a subscriber.
  • Edited the comment, adding some unrelated text at the end (e.g., fixing a typo).
    • Before change: @dog re-added as subscriber.
    • After change: no re-add.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Don't re-mention users for comment edits.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 13 2016, 6:31 PM
This revision was automatically updated to reflect the committed changes.