Homestyx hydra
Diffusion hydra 2085716da6d1

Make dashboard arrange actions (move, add, remove) work again after read…

Description

Make dashboard arrange actions (move, add, remove) work again after read locking from "chaos reduction"

Summary:
See PHI385. Ref T13054. Ref T13083. The dashboard "arrange" operations (add, remove, move) rely on doing $dashboard->setThing(...) and then applying transactions.

This no longer works after the read locking change from T13054. To make this function again, just add an explicit save() after layout adjustment. This should be more nuanced eventually, but all arrange operations are nonfunctional in a corrupting way at HEAD of master/stable, so stop the bleeding first.

Test Plan:

  • Created new empty and template dashboards.
  • Moved panels.
  • Added new and existing panels.
  • Removed panels.

Maniphest Tasks: T13083, T13054

Differential Revision: https://secure.phabricator.com/D19123

Details

Provenance
epriestleyAuthored on Feb 20 2018, 8:06 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:2fb266de7c1a: Fix some of the most obvious bugs in fact generation from Maniphest tasks
Branches
Unknown
Tags
Unknown

Event Timeline