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)
Thu, Apr 25, 12:11 AM
Unknown Object (File)
Fri, Apr 19, 11:41 PM
Unknown Object (File)
Thu, Apr 18, 12:51 AM
Unknown Object (File)
Fri, Apr 12, 1:18 AM
Unknown Object (File)
Thu, Apr 11, 8:28 AM
Unknown Object (File)
Feb 28 2024, 1:38 PM
Unknown Object (File)
Feb 12 2024, 5:55 AM
Unknown Object (File)
Jan 11 2024, 3:39 AM
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