Page MenuHomePhabricator

Legalpad - add a view signatures page
ClosedPublic

Authored by btrahan on Jan 17 2014, 12:35 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 4, 11:53 PM
Unknown Object (File)
Wed, Sep 4, 8:00 PM
Unknown Object (File)
Sat, Aug 31, 3:15 PM
Unknown Object (File)
Thu, Aug 29, 9:25 PM
Unknown Object (File)
Wed, Aug 28, 5:53 AM
Unknown Object (File)
Mon, Aug 26, 5:30 AM
Unknown Object (File)
Sun, Aug 25, 7:11 AM
Unknown Object (File)
Sat, Aug 24, 10:22 AM

Details

Reviewers
epriestley
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP4e6390774fac: Legalpad - add a view signatures page
Summary

...needs to add a LegalpadDocumentSignatureQuery class to get this done, which is also re-deployed everywhere we were issuing raw queries. Ref T3116.

Test Plan

viewed some signatures. Verified color and footer icons showed up how I wanted them to.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley added inline comments.
src/applications/legalpad/controller/LegalpadDocumentSignatureVerificationController.php
24โ€“36

Will $user (i.e., a logged out user) be able to see the signature? I think they won't, so this will break email signatures?

src/applications/legalpad/controller/LegalpadDocumentSignatureVerificationController.php
24โ€“36

ah, nice catch. i'll just issue a raw query with a comment...

btrahan updated this revision to Unknown Object (????).Jan 17 2014, 7:37 PM

fix not logged in case for verifying sigs