Page MenuHomePhabricator
Differential D20217

Move Harbormaster Build Plans to modular transactions
ClosedPublic
Actions

Authored by epriestley on Feb 26 2019, 1:17 PM.
Tags
None
Referenced Files
F13204412: D20217.diff
Wed, May 15, 12:33 AM
F13199219: D20217.id.diff
Mon, May 13, 12:57 PM
F13187349: D20217.diff
Sat, May 11, 4:28 AM
Unknown Object (File)
Tue, May 7, 7:35 AM
Unknown Object (File)
Mon, May 6, 7:54 AM
Unknown Object (File)
Fri, May 3, 6:57 AM
Unknown Object (File)
Fri, May 3, 1:32 AM
Unknown Object (File)
Fri, Apr 26, 7:43 PM
View All 89 Files
Subscribers
Restricted Owners Package

Details

Reviewers
amckinley
Maniphest Tasks
T13258: Plans: Harbormaster Build Plan Flags ("Does Not Block Promotion", etc)
Commits
rPf6ed873f1728: Move Harbormaster Build Plans to modular transactions
Summary

Depends on D20216. Ref T13258. Bland infrastructure update to prepare for bigger things.

Test Plan

Created and edited a build plan.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 26 2019, 1:17 PM
Owners added a subscriber: Restricted Owners Package.Feb 26 2019, 1:17 PM
epriestley added inline comments.Feb 26 2019, 1:18 PM
src/applications/harbormaster/storage/configuration/HarbormasterBuildPlanTransaction.php
44

function getColor() { ... return parent::getIcon(); } as an interesting implementation choice.

Harbormaster completed remote builds in B22128: Diff 48266.Feb 26 2019, 1:19 PM
epriestley requested review of this revision.Feb 26 2019, 1:19 PM
epriestley added a child revision: D20218: Provide "harbormaster.buildplan.edit" in the API.Feb 26 2019, 1:23 PM
amckinley accepted this revision.Feb 26 2019, 3:59 PM
This revision is now accepted and ready to land.Feb 26 2019, 3:59 PM
Closed by commit rPf6ed873f1728: Move Harbormaster Build Plans to modular transactions (authored by epriestley). · Explain WhyFeb 28 2019, 3:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 48278

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/harbormaster/controller/HarbormasterPlanDisableController.php

Loading...
View Options

src/applications/harbormaster/editor/HarbormasterBuildPlanEditEngine.php

Loading...
View Options

src/applications/harbormaster/editor/HarbormasterBuildPlanEditor.php

Loading...
View Options

src/applications/harbormaster/storage/configuration/HarbormasterBuildPlan.php

Loading...
View Options

src/applications/harbormaster/storage/configuration/HarbormasterBuildPlanTransaction.php

Loading...
View Options

src/applications/harbormaster/xaction/plan/HarbormasterBuildPlanNameTransaction.php

Loading...
View Options

src/applications/harbormaster/xaction/plan/HarbormasterBuildPlanStatusTransaction.php

Loading...
View Options

src/applications/harbormaster/xaction/plan/HarbormasterBuildPlanTransactionType.php

Loading...
Phabricator · Built by Phacility