Page MenuHomePhabricator

Tune document details in Legalpad
ClosedPublic

Authored by epriestley on Aug 23 2015, 2:34 PM.
Tags
None
Referenced Files
F13150063: D13982.diff
Sat, May 4, 1:20 PM
Unknown Object (File)
Fri, May 3, 8:37 AM
Unknown Object (File)
Mon, Apr 29, 2:05 AM
Unknown Object (File)
Sun, Apr 28, 9:28 PM
Unknown Object (File)
Sat, Apr 27, 10:33 PM
Unknown Object (File)
Thu, Apr 25, 2:28 AM
Unknown Object (File)
Fri, Apr 19, 12:36 PM
Unknown Object (File)
Wed, Apr 17, 12:08 PM
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
Branch
legal1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7744
Build 8521: [Placeholder Plan] Wait for 30 Seconds
Build 8520: arc lint + arc unit

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.