Page MenuHomePhabricator

Remove rows for personal saved builtin queries
ClosedPublic

Authored by epriestley on Aug 24 2017, 8:19 PM.
Tags
None
Referenced Files
F16568894: D18464.id44367.diff
Tue, Jun 17, 3:14 AM
F16450098: D18464.diff
Mon, Jun 16, 2:36 PM
Unknown Object (File)
Mon, Jun 2, 8:39 PM
Unknown Object (File)
Sat, May 31, 11:01 AM
Unknown Object (File)
Thu, May 29, 3:02 AM
Unknown Object (File)
Tue, May 20, 5:30 PM
Unknown Object (File)
Apr 27 2025, 3:06 AM
Unknown Object (File)
Apr 26 2025, 9:36 PM
Subscribers
None

Details

Summary

Ref T12956. After this change, individual users will no longer be able to modify builtin queries on a user-by-user basis: they will always appear at the bottom of the list, under their personal queries, and can only be managed by administrators.

To support this, clean up the old rows which could be hanging around from before: delete any personal saved queries where the saved query is a builtin query.

To ease this transition, try to pin the query we're deleting if the user had reordered things to put it on top.

Test Plan
  • Ran the migration, saw no changes in the UI but fewer rows.
  • Went back to master, reordered queries to put a builtin one on top.
  • Ran the migration.
  • Saw that builtin one drop to the bottom (since it can't be on top anymore) but be pinned, preserving the behavior of /maniphest/.

Diff Detail

Repository
rP Phabricator
Branch
item4
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 18146
Build 24390: Run Core Tests
Build 24389: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Aug 24 2017, 8:52 PM
This revision was automatically updated to reflect the committed changes.