Page MenuHomePhabricator

Implement open() and closed() Maniphest tokenizer status functions, plus cleanup
ClosedPublic

Authored by epriestley on Apr 23 2015, 1:49 PM.
Tags
None
Referenced Files
F12822905: D12530.id30077.diff
Thu, Mar 28, 7:35 AM
Unknown Object (File)
Feb 21 2024, 2:10 AM
Unknown Object (File)
Feb 14 2024, 12:19 PM
Unknown Object (File)
Feb 13 2024, 11:47 PM
Unknown Object (File)
Feb 13 2024, 11:47 PM
Unknown Object (File)
Feb 13 2024, 11:40 PM
Unknown Object (File)
Feb 7 2024, 10:08 PM
Unknown Object (File)
Feb 3 2024, 3:28 AM
Subscribers
Tokens
"Haypence" token, awarded by chad.

Details

Summary

Ref T4100.

  • Make it easy to choose all open or closed tasks.
  • Make "special" tokenizers composable.
  • Get viewer() generating documentation properly.
Test Plan
  • Ran queries with new tokens.
  • Browsed new tokens.
  • Viewed docs on new tokens.
  • Used plain status tokens.

Diff Detail

Repository
rP Phabricator
Branch
search8
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5472
Build 5490: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Implement open() and closed() Maniphest tokenizer status functions, plus cleanup.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

One minor thing is that the "open status" default icon (an unchecked box) looks kind of like a unicode error when shown in context:

Screen_Shot_2015-04-23_at_7.25.43_AM.png (103×467 px, 18 KB)

I don't know if that will bother anyone but me.

Also, how useful do you guys think min() and max() functions would be for priority? I'm inclined to wait for users to request them since they seem less clearly useful than open() and closed(), but if you'd definitely use them I can build them now.

For example, you could use min(normal) to select all tasks with priority normal or higher.

It's not something I would use day to day, but I could see its use in bigger companies (ones that highly customize priority levels).

I like min() and max() but wouldn't use them much here and can easily work around when I do.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 23 2015, 5:31 PM
This revision was automatically updated to reflect the committed changes.