Page MenuHomePhabricator

In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules
Closed, ResolvedPublic

Description

Hi all,

On our installation of phabricator which has neither legalpad nor harbormaster installed, when creating a herald rule both of these options show up: https://www.dropbox.com/s/p0z6r5eqj0awgwq/Screenshot%202015-04-27%2011.41.59.png?dl=0

Herald should not offer the option to require features from applications which are non-functional.

(I'm uncertain about what to do with rules set up prior to them being uninstalled)

Event Timeline

eadler raised the priority of this task from to Needs Triage.
eadler updated the task description. (Show Details)
eadler added projects: Herald, Applications.
eadler added a project: User Delight.
eadler added a subscriber: eadler.
epriestley triaged this task as Wishlist priority.Apr 27 2015, 8:49 PM
epriestley removed a project: User Delight.
yurotet renamed this task from Herald should not show actions for uninstalled applications to daglkasdjl fkefdsfe .Dec 10 2015, 10:50 AM
yurotet added a subscriber: yurotet.
johnny-bit renamed this task from daglkasdjl fkefdsfe to Herald should not show actions for uninstalled applications.Dec 10 2015, 11:04 AM
johnny-bit edited subscribers, added: johnny-bit; removed: yurotet.

Two aspects here, one or both of which may already be partially or fully implemented:

  • HeraldCreate Herald Rule should not show any options for applications which aren't installed for the viewer.
  • Application-specific conditions (like "User's projects include") and actions (like "Require Legalpad Signature") should not be available if the corresponding applications are not installed (do a global test here, not a viewer test, because the viewer may be examining or editing another user's rule, and that other user may have access).

Oh, I think I made up point (1) above -- that's already the behavior.

Point (2) is worth fixing at some point, but not relevant to T7939.

epriestley renamed this task from Herald should not show actions for uninstalled applications to In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules.Jan 26 2018, 5:33 PM
epriestley claimed this task.

D20808 fixes the two original cases (Harbormaster and Legalpad). There are probably more fields or actions which could be configured to be sometimes-unavailable, but these are probably the major ones.