Page MenuHomePhabricator

Herald - add support for task priority
ClosedPublic

Authored by btrahan on Feb 6 2014, 7:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 2, 3:06 PM
Unknown Object (File)
Thu, Aug 29, 2:57 AM
Unknown Object (File)
Mon, Aug 26, 3:32 AM
Unknown Object (File)
Sun, Aug 25, 5:08 PM
Unknown Object (File)
Sun, Aug 25, 1:35 PM
Unknown Object (File)
Mon, Aug 19, 6:01 PM
Unknown Object (File)
Sun, Aug 18, 11:33 PM
Unknown Object (File)
Tue, Aug 13, 6:34 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.