Page MenuHomePhabricator

Allow "drydock.blueprint.edit" to create blueprints
ClosedPublic

Authored by epriestley on Dec 26 2017, 6:24 PM.
Tags
None
Referenced Files
F15464833: D18849.id45215.diff
Wed, Apr 2, 12:53 PM
F15436877: D18849.id45215.diff
Tue, Mar 25, 4:42 PM
F15430607: D18849.id.diff
Mon, Mar 24, 8:26 AM
F15418044: D18849.id45215.diff
Thu, Mar 20, 8:24 PM
F15417873: D18849.id45228.diff
Thu, Mar 20, 7:05 PM
F15411257: D18849.id.diff
Wed, Mar 19, 8:57 AM
F15408244: D18849.diff
Tue, Mar 18, 9:51 PM
F15403963: D18849.id45215.diff
Tue, Mar 18, 5:41 AM
Subscribers
Restricted Owners Package

Details

Summary

Depends on D18848. Ref PHI243. This puts a bit of logic up front to figure out the blueprint type before we actually start editing it.

This implementation is a little messy but it keeps the API clean. Eventually, the implementation could probably go in the TransactionTypes so more code is shared, but I'd like to wait for a couple more of these first.

This capability probably isn't too useful, but just pays down a bit of technical debt from the caveat introduced in D18822.

Test Plan
  • Created a new blueprint with the API.
  • Tried to create a blueprint without a "type" (got a helpful error).
  • Created and edited blueprints via the web UI.
  • Tried to change the "type" of an existing blueprint (got a helpful error).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Dec 26 2017, 6:24 PM
This revision is now accepted and ready to land.Jan 2 2018, 7:03 PM
This revision was automatically updated to reflect the committed changes.