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.
Details
Details
- Reviewers
epriestley - Maniphest Tasks
- T10145: Adding a panel to a dashboard multiple times
- Commits
- rP5556f0e45a52: Don't allow duplicate panels on dashboards
Try to add the same panel twice, get error. Add panel normally fine, move panels fine, edit panels fine.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- dupe-panels (branched from master)
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 15625 Build 20622: Run Core Tests Build 20621: arc lint + arc unit
Event Timeline
Comment Actions
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.