Page MenuHomePhabricator

Herald - add support for task priority
ClosedPublic

Authored by btrahan on Feb 6 2014, 7:35 PM.
Tags
None
Referenced Files
F17747313: D8156.id18454.diff
Mon, Jul 21, 9:50 PM
F17743980: D8156.diff
Mon, Jul 21, 2:59 PM
F17741556: D8156.id18455.diff
Mon, Jul 21, 9:31 AM
F17724307: D8156.diff
Fri, Jul 18, 12:43 PM
Unknown Object (File)
May 18 2025, 8:14 PM
Unknown Object (File)
May 5 2025, 6:00 AM
Unknown Object (File)
May 5 2025, 6:00 AM
Unknown Object (File)
May 5 2025, 6:00 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

Repository
rP Phabricator
Branch
heraldtaskpri
Lint
Lint Passed
Unit
Tests Passed

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.