Page MenuHomePhabricator
Differential D18014

Migrate Project workboard background color to modular transactions
ClosedPublic
Actions

Authored by amckinley on May 24 2017, 7:54 PM.
Tags
None
Referenced Files
F19945128: D18014.id43327.diff
Thu, Apr 16, 3:23 PM
F19945126: D18014.id43327.diff
Thu, Apr 16, 3:23 PM
F19945061: D18014.id43327.diff
Thu, Apr 16, 3:20 PM
F19944795: D18014.id43328.diff
Thu, Apr 16, 1:25 PM
F19504498: D18014.id43328.diff
Jan 9 2026, 5:14 AM
F19504497: D18014.id43327.diff
Jan 9 2026, 5:14 AM
F19504495: D18014.id.diff
Jan 9 2026, 5:14 AM
F19504445: D18014.id43328.diff
Jan 9 2026, 5:11 AM
View All Files
Subscribers
epriestley

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T12673: Update Projects for Modular Transactions
Commits
rP88466addee26: Migrate Project workboard background color to modular transactions
Summary

Removes now-unused method as well. Fixes T12673.

Test Plan

UI fiddling.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley created this revision.May 24 2017, 7:54 PM
Herald added a subscriber: epriestley. · View Herald TranscriptMay 24 2017, 7:54 PM
epriestley accepted this revision.May 24 2017, 7:54 PM

v. nice

This revision is now accepted and ready to land.May 24 2017, 7:54 PM
Harbormaster completed remote builds in B17239: Diff 43327.May 24 2017, 7:55 PM
amckinley added inline comments.May 24 2017, 7:55 PM
src/applications/project/editor/PhabricatorProjectTransactionEditor.php
56–60

I'm pretty sure (and a quick test seemed to confirm) that this code isn't needed with the way modular transactions work.

Closed by commit rP88466addee26: Migrate Project workboard background color to modular transactions (authored by amckinley, committed by amckinley). · Explain WhyMay 24 2017, 7:56 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 43328

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/project/controller/PhabricatorProjectBoardBackgroundController.php

Loading...
View Options

src/applications/project/editor/PhabricatorProjectTransactionEditor.php

Loading...
View Options

src/applications/project/storage/PhabricatorProjectTransaction.php

Loading...
View Options

src/applications/project/xaction/PhabricatorProjectWorkboardBackgroundTransaction.php

Loading...
Phabricator · Built by Phacility