Page MenuHomePhabricator

Tune document details in Legalpad
ClosedPublic

Authored by epriestley on Aug 23 2015, 2:34 PM.
Tags
None
Referenced Files
F13090594: D13982.diff
Thu, Apr 25, 2:28 AM
Unknown Object (File)
Fri, Apr 19, 12:36 PM
Unknown Object (File)
Wed, Apr 17, 12:08 PM
Unknown Object (File)
Thu, Apr 11, 9:52 AM
Unknown Object (File)
Wed, Apr 10, 7:58 PM
Unknown Object (File)
Sun, Apr 7, 4:05 AM
Unknown Object (File)
Wed, Apr 3, 11:30 AM
Unknown Object (File)
Wed, Apr 3, 11:30 AM
Subscribers
None

Details

Summary

Fixes T9245. These picked up some possibly-confusing metadata, like in the screenshot on T9245 where "Subscribers" appears in the middle of the page for no obvious reason.

  • Make these pages a little cleaner by removing elements which aren't important for signing agreements.
  • Use the last time the actual document text was updated as the modification time, not the last time the "Document" object was modified. The latter will change for trivial things like altering the view/edit policy, but that could be confusing if you see that a TOS was "last updated yesterday" but can't figure out what actually changed (since nothing changed).
Test Plan

Viewed signature page for a document.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 33724.
epriestley retitled this revision from to Tune document details in Legalpad.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Fix comment wording.
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 23 2015, 3:01 PM
This revision was automatically updated to reflect the committed changes.