Page MenuHomePhabricator

Migrate Project workboard to modular transactions
ClosedPublic

Authored by amckinley on May 23 2017, 7:07 PM.
Tags
None
Referenced Files
F19554342: D18004.id43303.diff
Thu, Jan 29, 8:52 PM
F19396558: D18004.id43301.diff
Dec 27 2025, 9:57 AM
F18855336: D18004.id43303.diff
Nov 1 2025, 4:55 AM
F18841778: D18004.id.diff
Oct 28 2025, 11:13 AM
F18841511: D18004.diff
Oct 28 2025, 9:40 AM
F18767507: D18004.diff
Oct 7 2025, 10:54 PM
F18737867: D18004.id43301.diff
Oct 1 2025, 1:04 PM
F18629412: D18004.diff
Sep 16 2025, 8:42 AM
Subscribers

Details

Summary

This was interesting, because there were a mix of callsites using transactions and others that just set the property on the Project object. I made everything consistent in using transactions to change this property. I also found an implementation of getTitle() that I don't think is ever being invoked since shouldHide() is returning true, but I migrated it anyway.

Test Plan

Unit tests pass + enabling/disabling workboards (and importing).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I think these just didn't get converted when we swapped to transactions originally, your rewrites look correct to me.

This revision is now accepted and ready to land.May 23 2017, 7:18 PM
This revision was automatically updated to reflect the committed changes.