Page MenuHomePhabricator

Update rendering of policy edit transactions in Applications
ClosedPublic

Authored by epriestley on Sep 12 2019, 2:03 PM.
Tags
None
Referenced Files
F13245355: D20801.diff
Thu, May 23, 5:58 AM
F13243387: D20801.diff
Thu, May 23, 3:52 AM
F13223401: D20801.diff
Sun, May 19, 4:34 AM
F13205575: D20801.diff
Wed, May 15, 2:10 AM
F13189488: D20801.diff
Sat, May 11, 6:06 AM
Unknown Object (File)
Tue, May 7, 3:09 PM
Unknown Object (File)
Tue, May 7, 9:42 AM
Unknown Object (File)
Mon, May 6, 9:20 PM
Subscribers
None

Details

Summary

Ref T13411. Since circa D19829, transactions have rendered policy changes in a modern way, notably making "Custom Policy" clickable to show the policy rules.

Edit transactions in Applications still use a separate, older approach to render policies. This produces policy renderings which don't use modern quoting rules and don't link in a modern way.

Make Applications use the same rendering code that other transactions (like normal edit/view edits) use.

Test Plan

Edited policies in Applications, saw more useful transactions in the log. Clicked "Custom Policy" in the transaction log and got a useful explanation of the policy.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Sep 12 2019, 4:32 PM
This revision was automatically updated to reflect the committed changes.