Page MenuHomePhabricator

Give EditEngine a Conduit-specific initialization pathway for objects
ClosedPublic

Authored by epriestley on Dec 26 2017, 5:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 9:22 PM
Unknown Object (File)
Wed, Dec 18, 1:21 AM
Unknown Object (File)
Mon, Dec 9, 5:50 PM
Unknown Object (File)
Sun, Dec 8, 9:49 PM
Unknown Object (File)
Fri, Dec 6, 5:30 PM
Unknown Object (File)
Fri, Dec 6, 4:03 PM
Unknown Object (File)
Thu, Nov 28, 2:48 AM
Unknown Object (File)
Wed, Nov 27, 1:15 PM
Subscribers
None

Details

Summary

Depends on D18845. See PHI243 for context and more details.

Briefly, some objects need a "type" transaction (or something similar) very early on, and we can't generate their fields until we know the object type. Drydock blueprints are an example: a blueprint's fields depend on the blueprint's type.

In web interfaces, the workflow just forces the user to select a type first. For Conduit workflows, I think the cleanest approach is to proactively extract and apply type information before processing the request. This will make the implementation a little messier, but the API cleaner.

An alternative is to add more fields to the API, like a "type" field. This makes the implementation cleaner, but the API messier. I think we're better off favoring a cleaner API here.

This change just makes it possible for DrydockBlueprintEditEngine to look at the incoming transactions and extract a "type"; it doesn't actually change any behavior.

Test Plan

Performed edits via API, but this change doesn't alter any behavior.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable