Page MenuHomePhabricator

Give policy name rendering explicit "text name", "capability link", and "transaction link" pathways
ClosedPublic

Authored by epriestley on Sep 12 2019, 2:46 PM.
Tags
None
Referenced Files
F19152556: D20804.id49608.diff
Thu, Dec 11, 3:34 AM
F19107998: D20804.diff
Dec 8 2025, 3:04 PM
F19095214: D20804.id49600.diff
Dec 4 2025, 4:23 AM
F19083058: D20804.id49608.diff
Dec 2 2025, 12:29 PM
F19034992: D20804.diff
Nov 25 2025, 3:51 PM
F19019242: D20804.diff
Nov 23 2025, 1:58 PM
F19000981: D20804.diff
Nov 21 2025, 1:23 AM
F18985205: D20804.id49600.diff
Nov 17 2025, 12:55 PM
Subscribers
None

Details

Summary

Ref T13411. This cleans up policy name rendering. We ultimately render into three contexts:

  • Plain text contexts, like bin/policy show.
  • Transaction contexts, where we're showing a policy change. In these cases, we link some policies (like project policies and custom policies) but the links go directly to the relevant object or a minimal explanation of the change. We don't link policies like "All Users".
  • Capability contexts, where we're describing a capability, like "Can Push" or cases in Applicaitons. In these cases, we link all policies to the full policy explanation flow.
Test Plan
  • Used bin/policy show to examine the policy of an object with a project policy, no longer saw HTML.
  • Viewed the transaction logs of Applications (ModularTransactions) and Tasks (not ModularTransactions) with policy edits, including project and custom policies.
  • Clicked "Custom Policy" in both logs, got consistent dialogs.
  • Viewed application detail pages, saw all capabities linked to explanatory capability dialogs. The value of having this dialog is that the user can get a full explanation of special rules even if the policy is something mundane like "All Users".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable