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)
Mon, Nov 18, 4:42 AM
Unknown Object (File)
Fri, Nov 15, 10:09 AM
Unknown Object (File)
Sun, Nov 10, 2:36 AM
Unknown Object (File)
Sat, Nov 9, 5:16 PM
Unknown Object (File)
Sat, Nov 2, 4:09 AM
Unknown Object (File)
Sat, Nov 2, 4:09 AM
Unknown Object (File)
Sat, Nov 2, 4:09 AM
Unknown Object (File)
Sat, Nov 2, 4:09 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