Page MenuHomePhabricator

Remove property "id" from Workboard View controller
ClosedPublic

Authored by epriestley on Jun 29 2019, 9:20 PM.
Tags
None
Referenced Files
F14398859: D20627.diff
Sun, Dec 22, 3:07 PM
Unknown Object (File)
Tue, Dec 17, 10:39 AM
Unknown Object (File)
Tue, Dec 17, 4:50 AM
Unknown Object (File)
Fri, Dec 13, 8:52 AM
Unknown Object (File)
Fri, Dec 13, 8:52 AM
Unknown Object (File)
Wed, Dec 11, 7:31 PM
Unknown Object (File)
Wed, Dec 11, 4:21 AM
Unknown Object (File)
Thu, Dec 5, 3:29 AM
Subscribers
None

Details

Summary

Depends on D20626. Ref T4900. On this controller, "id" is a separate property, but serves little purpose and complicates separating state management. Remove it.

Test Plan

Bulk edited a column, managed filters, did show/hide on columns, edited a column.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable