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)
Thu, Dec 26, 11:19 PM
Unknown Object (File)
Sat, Dec 21, 5:38 AM
Unknown Object (File)
Thu, Dec 12, 6:20 AM
Unknown Object (File)
Sat, Dec 7, 9:54 PM
Unknown Object (File)
Sat, Dec 7, 5:21 AM
Unknown Object (File)
Sat, Nov 30, 3:10 AM
Unknown Object (File)
Nov 26 2024, 3:51 PM
Unknown Object (File)
Nov 26 2024, 3:51 PM

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.