Page MenuHomePhabricator

Limit damage caused by bad panels on dashboards you can manage
ClosedPublic

Authored by epriestley on Feb 4 2017, 2:33 AM.
Tags
None
Referenced Files
F19824562: D17311.diff
Sun, Mar 8, 12:37 AM
F19714740: D17311.id41635.diff
Thu, Feb 12, 2:31 AM
F19680087: D17311.id41635.diff
Sun, Feb 8, 6:52 PM
F19615702: D17311.diff
Feb 4 2026, 1:02 AM
F19522738: D17311.diff
Jan 17 2026, 10:03 AM
F19517347: D17311.diff
Jan 15 2026, 10:03 AM
F19515314: D17311.diff
Jan 14 2026, 10:06 AM
F19077323: D17311.diff
Dec 1 2025, 2:50 PM
Subscribers
None

Details

Summary

Fixes T12203. If you tried to manage a dashboard which had a panel you can't see, we'd try to render bogus actions for it and fatal.

Instead, for the moment, survive. Presumably we'll ship a real fix for this in the next release or so, and tackle T10612 / T10145, which I think are closely related.

Test Plan

Screen Shot 2017-02-03 at 6.31.21 PM.png (959×1 px, 160 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable