Home GnuPG

Allow users who can edit a dashboard to remove invalid / restricted panels
f64edb993f68Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Allow users who can edit a dashboard to remove invalid / restricted panels

Summary:
Ref T12207. Currently, to remove a panel from a dashboard, it must be a valid panel which you can see.

Instead, only require that the panel PHID actually be listed somewhere in the dashboard's internal list of panels.

This interacts with the "multiple instances of a panel" issue described in some more depth in T12207. In particular:

  • Currently, you can sort of add multiple copies of a panel to a dashboard, sometimes? Maybe?
  • This leads to great tragedy.

This doesn't fix up the workflow with respect to multiple copies of a panel. We still remove by panel PHID (not by column/position or internal ID) so if a dashboard has multiple copies of the same panel for some reason, I think this workflow removes one of them arbitrarily (at best) or perhaps does something worse. I'm just treating this behavior as undefined for the moment.

Test Plan:

  • Removed an invalid/hidden panel from a dashboard as a user with permission to edit that dashboard.
  • Tried to remove a made-up panel with a totally bogus PHID, got 404'd.
  • Viewed a dashboard with a restricted panel.
  • Put a hidden panel inside a tab panel, viewed it as a user who could not see it and a user who could.

Reviewers: chad

Reviewed By: chad

Subscribers: swisspol

Maniphest Tasks: T12207

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

Details

Provenance
epriestley <git@epriestley.com>Authored on Feb 5 2017, 1:06 AM
Parents
rPHABd2c4d7d961b9: Clarify linking to comments in Remarkup Guide
Branches
Unknown
Tags
Unknown

Event Timeline