Differential D19746

Make the Harbormaster build plan edit engine forms configurable
AcceptedPublic
Actions

Authored by epriestley on Oct 16 2018, 7:48 PM.

Details

Reviewers

Maniphest Tasks

T13210: Plans: 2018 Week 41-44 Bonus Content

Summary

See PHI933. Ref T13210. Harbormaster build plans were convereted to EditEngine some time ago (see D15357) but the forms were not made configurable.

I believe this was just to avoid unnecessary complexity, and there's no reason not to allow configuration of these forms. In particular, support for form configuration allows you to set alternate default view/edit policies and add instructions.

PHI933 is a specific request for changing the default policies, and this is a reasonable approach to providing support.

Test Plan

Created a new build plan, edited build plan forms, changed form name, chagned form defaults, saw new defaults/name reflected in UI.

Diff Detail

Repository

rP Phabricator

Branch

plan1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 22577
Build 30926: Run Core Tests
Build 30925: arc lint + arc unit

Event Timeline

epriestley created this revision.Oct 16 2018, 7:48 PM

Owners added a subscriber: Restricted Owners Package.Oct 16 2018, 7:48 PM

Harbormaster completed remote builds in B21016: Diff 47173.Oct 16 2018, 7:49 PM

epriestley requested review of this revision.Oct 16 2018, 7:49 PM

This is slightly more involved than I initially believed, we have a "Default Edit Policy" and "Default View Policy" which should probably be nuked now and PHI933 also has a sub-request for an "Author" object policy.

Let's move forward with this after all: although we need some more work to fix the "Default XYZ Policy" stuff, this is the best path forward for PHI1204 (which requests that build plan behaviors have configurable defaults).

Harbormaster completed remote builds in B22577: Diff 48707.Apr 12 2019, 11:21 PM

amckinley accepted this revision.Apr 17 2019, 5:33 PM

This revision is now accepted and ready to land.Apr 17 2019, 5:33 PM