Page MenuHomePhabricator

Herald - add support for task priority
ClosedPublic

Authored by btrahan on Feb 6 2014, 7:35 PM.
Tags
None
Referenced Files
F14099470: D8156.id18455.diff
Tue, Nov 26, 11:45 AM
F14099469: D8156.id18454.diff
Tue, Nov 26, 11:45 AM
F14099468: D8156.id.diff
Tue, Nov 26, 11:45 AM
F14099394: D8156.diff
Tue, Nov 26, 11:33 AM
Unknown Object (File)
Mon, Nov 25, 2:04 PM
Unknown Object (File)
Thu, Nov 21, 1:41 PM
Unknown Object (File)
Wed, Nov 20, 8:42 PM
Unknown Object (File)
Sun, Nov 17, 10:23 AM

Details

Summary

adds a new FIELD and a new VALUE to support this. Slightly dodgy because priorities do not have phids so we have to special case how we handle this in a few spots. Ref T4294.

Test Plan

made a new rule to get cc'd on unbreak now and wishlist tasks. verified got cc'd correctly and not cc'd correctly.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

This looks perfectly reasonable to me.

At some point I want to make the typeahead datasource modular so each application can define result types and it's easier to add new stuff, but I'll probably tackle that when I get back around to that diff which added icons to results.