Page MenuHomePhabricator

Fix overzealous subscribing in EditEngine
ClosedPublic

Authored by epriestley on Dec 16 2015, 12:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 9:45 PM
Unknown Object (File)
Sun, Dec 1, 12:13 PM
Unknown Object (File)
Sun, Dec 1, 7:13 AM
Unknown Object (File)
Sun, Dec 1, 7:13 AM
Unknown Object (File)
Sun, Dec 1, 7:13 AM
Unknown Object (File)
Nov 6 2024, 2:12 AM
Unknown Object (File)
Nov 5 2024, 11:32 PM
Unknown Object (File)
Nov 1 2024, 2:55 PM
Subscribers

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP1d72c97fc9f6: Fix overzealous subscribing in EditEngine
Summary

See T9905#148799. The CommentEditField generated empty comment transactions; these are dropped later, but before they are dropped they would trigger implicit CCs.

The implicit CC rule should probably be narrower, but we shouldn't be generating these transactions in the first place.

Test Plan

No longer implicitly CC'd on a task when doing something minor like changing projects.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix overzealous subscribing in EditEngine.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a subscriber: avivey.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 16 2015, 12:15 AM
This revision was automatically updated to reflect the committed changes.