Page MenuHomePhabricator

Add a "documents I've signed" view to Legalpad
ClosedPublic

Authored by epriestley on Jun 28 2014, 8:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 16, 2:48 AM
Unknown Object (File)
Thu, Sep 12, 3:17 AM
Unknown Object (File)
Wed, Sep 4, 10:16 AM
Unknown Object (File)
Sat, Aug 31, 7:43 PM
Unknown Object (File)
Tue, Aug 27, 11:49 PM
Unknown Object (File)
Tue, Aug 27, 1:17 PM
Unknown Object (File)
Tue, Aug 27, 12:19 AM
Unknown Object (File)
Sun, Aug 25, 1:09 PM
Subscribers

Details

Reviewers
chad
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP5242fb057251: Add a "documents I've signed" view to Legalpad
Summary

Ref T3116. Allow documents to be queried for ones the viewer has signed, and make this the default view.

This also relaxes the versioning stuff a little bit, and stops invalidating signatures on older versions of documents. While I think we should do that eventually, it should be more explicit and have better coordination in the UI. For now, we'll track and show older signatures, but not invalidate them.

I imagine eventually differentiating between "minor edits" (typo / link fixes, for example) and major edits which actually require re-signature.

Test Plan

Screen_Shot_2014-06-28_at_1.51.56_PM.png (898×1 px, 120 KB)

Diff Detail

Repository
rP Phabricator
Branch
legal6
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/policy/rule/PhabricatorPolicyRuleLegalpadSignature.php:18XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 1448
Build 1448: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Add a "documents I've signed" view to Legalpad.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: Restricted Maniphest Task.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 28 2014, 10:04 PM
epriestley updated this revision to Diff 23458.

Closed by commit rP5242fb057251 (authored by @epriestley).