Page MenuHomePhabricator

Split Repository EditEngine form into smaller pages
ClosedPublic

Authored by epriestley on May 2 2016, 12:40 PM.
Tags
None
Referenced Files
F14059154: D15832.diff
Sun, Nov 17, 5:54 PM
F14056773: D15832.id38139.diff
Sat, Nov 16, 10:21 PM
F14048739: D15832.diff
Thu, Nov 14, 9:30 AM
F14035127: D15832.diff
Sun, Nov 10, 4:23 AM
F14019134: D15832.diff
Tue, Nov 5, 9:35 PM
F13975752: D15832.id38139.diff
Oct 18 2024, 11:28 AM
Unknown Object (File)
Oct 11 2024, 4:36 PM
Unknown Object (File)
Sep 15 2024, 11:14 AM
Subscribers
None

Details

Summary

Ref T10748. This allows an EditEngine form to be broken up into pages.

This is less powerful than PHUIPagedFormView, because the pages are not sequential / stateful. Each form saves immediately once it's submitted, and can not take you to a new form or back/forward in a series of forms.

For example, you can't create a workflow where the user fills out 5 pages of information before we create an object, like the current repository workflow does.

However, the only place we've ever wanted to do this is repositories and it's fairly bad there, so I feel reasonably confident we aren't going to miss this in the future.

(We do "choose a type of service/repository/rule -> fill out one page of info" fairly often, but can do this without the full-power paging stuff.)

Test Plan
  • Created a repository usin the new Manage UI, filling out only a handful of fields.
  • Edited a repository using the new Manage UI.
  • All forms are now EditEngine forms offering paged views of the big huge underlying form:

Screen Shot 2016-05-02 at 5.39.15 AM.png (1×1 px, 127 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Split Repository EditEngine form into smaller pages.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.May 2 2016, 2:09 PM
This revision was automatically updated to reflect the committed changes.