Page MenuHomePhabricator

Give the workboard "default" workflows more modern state handling
ClosedPublic

Authored by epriestley on Jun 29 2019, 10:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 27, 6:36 PM
Unknown Object (File)
Fri, Dec 27, 3:57 PM
Unknown Object (File)
Tue, Dec 17, 2:18 PM
Unknown Object (File)
Tue, Dec 10, 4:37 PM
Unknown Object (File)
Tue, Dec 10, 6:37 AM
Unknown Object (File)
Sun, Dec 8, 6:27 AM
Unknown Object (File)
Nov 25 2024, 1:38 AM
Unknown Object (File)
Nov 21 2024, 11:38 AM
Subscribers
None

Details

Summary

Depends on D20628. Ref T4900. Currently, the "Save Current Order/Filter As Default" flows on workboards duplicate some state construction, and require parameters to be passed to them explicitly.

Now that state management is separate, they can reuse a bit more code and be made to look more like other similar controllers.

Test Plan
  • Changed the default order of a workboard.
  • Changed the default filter of a workboard.
  • Changed the order of a board to something non-default, then changed the filter, then saved the new filter as the default. Saw the modified order preserved and the modified filter removed, so I ended up in the right ("most correct") place: on the board, with my custom order in a URI parameter, and no filter URI parameter so I could see my new default filter behavior. This is an edge case that's not terribly important to get right, but we do get it right.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable