HomePhabricator

Support comments as an EditEngine field

Tags
None
Referenced Files
F976036: Screen Shot 2015-11-18 at 2.33.10 PM.png
Nov 23 2015, 12:27 AM
F976038: Screen Shot 2015-11-18 at 2.33.23 PM.png
Nov 23 2015, 12:27 AM
F976037: Screen Shot 2015-11-18 at 2.33.07 PM.png
Nov 23 2015, 12:27 AM
Subscribers

Description

Support comments as an EditEngine field

Summary:
Ref T9132. This adds an automatic "Comments" field, like the Subscribers/Projects/Policy fields.

The primary goals here are:

  • Allow users to make comments via Conduit.
  • In the future, get stackable action support.

As a side effect, this also allows you to put comments on create forms. This is a little silly but seems fine, and may be relevant on edit forms (which I'm not 100% sure how I want to handle yet). I've just hidden them by default for now.

Test Plan:

Screen Shot 2015-11-18 at 2.33.10 PM.png (145×1 px, 28 KB)

Screen Shot 2015-11-18 at 2.33.07 PM.png (103×1 px, 22 KB)

Screen Shot 2015-11-18 at 2.33.23 PM.png (1×1 px, 151 KB)

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T9132

Differential Revision: https://secure.phabricator.com/D14515

Details

Event Timeline

I wonder if instead of "Comment", "Justification" might have more use in certain cases? (why not both! "Comment/Justification")

See D14515 for discussion. This is mostly a side-effect / proof-of-concept. I hope to be able to remove it from the UI completely in a future version.