Page MenuHomePhabricator

Begin modularizing Herald field values
ClosedPublic

Authored by epriestley on Jul 9 2015, 4:06 PM.
Tags
None
Referenced Files
F13238258: D13613.id32971.diff
Tue, May 21, 6:23 PM
F13237792: D13613.id.diff
Tue, May 21, 2:57 PM
F13235348: D13613.id32895.diff
Tue, May 21, 4:45 AM
F13216480: D13613.id.diff
Sat, May 18, 1:09 AM
F13211874: D13613.diff
Fri, May 17, 6:11 AM
F13208475: D13613.id32971.diff
Thu, May 16, 3:50 PM
F13196952: D13613.diff
Sun, May 12, 11:42 PM
Unknown Object (File)
Fri, May 3, 7:54 AM
Subscribers

Details

Summary

Ref T8726. I'm primarily trying to modularize tokenizer values so we don't have to update JS to add a new one.

This is ultimately the blocker for "select" custom fields working in Herald.

This inches us toward that. I'm not modularizing conditions or control types in this round, but hope to end up with hard-coded conditions (which are highly general and very rarely change), hard-coded control types (which are also highly general and very rarely change) and completely modular fields and values (which have mid-to-low generality and change frequently).

Test Plan

Used UI to interact with "none", "text", and new-style "select" controls. No actual support for tokenizers yet.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Begin modularizing Herald field values.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jul 16 2015, 8:39 PM
This revision was automatically updated to reflect the committed changes.