Page MenuHomePhabricator

Don't allow duplicate panels on dashboards
ClosedPublic

Authored by chad on Feb 14 2017, 4:34 AM.
Tags
None
Referenced Files
F18816892: D17351.id41733.diff
Tue, Oct 21, 10:19 AM
F18774229: D17351.id.diff
Thu, Oct 9, 4:38 PM
F18769651: D17351.diff
Wed, Oct 8, 8:30 AM
F18733471: D17351.id.diff
Sep 30 2025, 10:35 PM
F18713339: D17351.id.diff
Sep 29 2025, 5:56 AM
F18508515: D17351.id.diff
Sep 5 2025, 2:51 AM
F18500587: D17351.diff
Sep 4 2025, 9:10 PM
F18198141: D17351.id41727.diff
Aug 17 2025, 10:52 PM
Subscribers

Details

Summary

Fixes T10145. I went with "don't add two panels", since panels are easy to create, I expect this to be a reasonable limit until we have better use cases.

Test Plan

Try to add the same panel twice, get error. Add panel normally fine, move panels fine, edit panels fine.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Cool. I think this is a good approach, at least for the foreseeable future: the use cases right now definitely feel pretty weak to me too.

This revision is now accepted and ready to land.Feb 14 2017, 12:02 PM
This revision was automatically updated to reflect the committed changes.